[tor-commits] [tor/master] Coverity complained that we were not checking this return value

nickm at torproject.org nickm at torproject.org
Mon Dec 22 16:13:18 UTC 2014


commit b94cb401d2185566e7fb52b78c5a5a6747987f99
Author: Nick Mathewson <nickm at torproject.org>
Date:   Mon Dec 22 11:13:11 2014 -0500

    Coverity complained that we were not checking this return value
---
 src/or/scheduler.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/or/scheduler.c b/src/or/scheduler.c
index d1a15aa..5b4dff2 100644
--- a/src/or/scheduler.c
+++ b/src/or/scheduler.c
@@ -156,7 +156,9 @@ scheduler_free_all(void)
   log_debug(LD_SCHED, "Shutting down scheduler");
 
   if (run_sched_ev) {
-    event_del(run_sched_ev);
+    if (event_del(run_sched_ev) < 0) {
+      log_warn(LD_BUG, "Problem deleting run_sched_ev");
+    }
     tor_event_free(run_sched_ev);
     run_sched_ev = NULL;
   }



More information about the tor-commits mailing list