[tor-commits] [tor/master] scan-build: Have clear_pending_onions walk the lists more obviously

nickm at torproject.org nickm at torproject.org
Fri Apr 25 05:30:23 UTC 2014


commit 1b3bddd013dab6d0aa8159e1690d944e226ed77f
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri Apr 18 21:17:40 2014 -0400

    scan-build: Have clear_pending_onions walk the lists more obviously
    
    As it stands, it relies on the fact that onion_queue_entry_remove
    will magically remove each onionskin from the right list.  This
    patch changes the logic to be more resilient to possible bugs in
    onion_queue_entry_remove, and less confusing to static analysis tools.
---
 src/or/onion.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/or/onion.c b/src/or/onion.c
index 30b983d..72571b7 100644
--- a/src/or/onion.c
+++ b/src/or/onion.c
@@ -329,12 +329,14 @@ onion_queue_entry_remove(onion_queue_t *victim)
 void
 clear_pending_onions(void)
 {
-  onion_queue_t *victim;
+  onion_queue_t *victim, *next;
   int i;
   for (i=0; i<=MAX_ONION_HANDSHAKE_TYPE; i++) {
-    while ((victim = TOR_TAILQ_FIRST(&ol_list[i]))) {
+    for (victim = TOR_TAILQ_FIRST(&ol_list[i]); victim; victim = next) {
+      next = TOR_TAILQ_NEXT(victim,next);
       onion_queue_entry_remove(victim);
     }
+    tor_assert(TOR_TAILQ_EMPTY(&ol_list[i]));
   }
   memset(ol_entries, 0, sizeof(ol_entries));
 }





More information about the tor-commits mailing list