[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[or-cvs] r10560: Reorder routerlist_assert_ok() and routerlist_check_bug_417( (tor/trunk/src/or)



Author: weasel
Date: 2007-06-11 15:46:23 -0400 (Mon, 11 Jun 2007)
New Revision: 10560

Modified:
   tor/trunk/src/or/routerlist.c
Log:
Reorder routerlist_assert_ok() and routerlist_check_bug_417(), so if the assert_ok triggers we know that our bug_417 is wrong

Modified: tor/trunk/src/or/routerlist.c
===================================================================
--- tor/trunk/src/or/routerlist.c	2007-06-11 15:26:02 UTC (rev 10559)
+++ tor/trunk/src/or/routerlist.c	2007-06-11 19:46:23 UTC (rev 10560)
@@ -1903,10 +1903,10 @@
   smartlist_add(rl->routers, ri);
   ri->routerlist_index = smartlist_len(rl->routers) - 1;
   router_dir_info_changed();
+  routerlist_check_bug_417();
 #ifdef DEBUG_ROUTERLIST
   routerlist_assert_ok(rl);
 #endif
-  routerlist_check_bug_417();
 }
 
 /** Adds the extrainfo_t <b>ei</b> to the routerlist <b>rl</b>, if there is a
@@ -1961,10 +1961,10 @@
   if (r == 0)
     extrainfo_free(ei);
 
+  routerlist_check_bug_417();
 #ifdef DEBUG_ROUTERLIST
   routerlist_assert_ok(rl);
 #endif
-  routerlist_check_bug_417();
   return r;
 }
 
@@ -1994,10 +1994,10 @@
   } else {
     routerinfo_free(ri);
   }
+  routerlist_check_bug_417();
 #ifdef DEBUG_ROUTERLIST
   routerlist_assert_ok(rl);
 #endif
-  routerlist_check_bug_417();
 }
 
 /** Remove an item <b>ri</b> from the routerlist <b>rl</b>, updating indices
@@ -2052,10 +2052,10 @@
       digestmap_remove(rl->desc_by_eid_map, ri->cache_info.extra_info_digest);
     routerinfo_free(ri);
   }
+  routerlist_check_bug_417();
 #ifdef DEBUG_ROUTERLIST
   routerlist_assert_ok(rl);
 #endif
-  routerlist_check_bug_417();
 }
 
 /** Remove a signed_descriptor_t <b>sd</b> from <b>rl</b>-\>old_routers, and
@@ -2174,10 +2174,10 @@
       ri_old->cache_info.signed_descriptor_len;
     routerinfo_free(ri_old);
   }
+  routerlist_check_bug_417();
 #ifdef DEBUG_ROUTERLIST
   routerlist_assert_ok(rl);
 #endif
-  routerlist_check_bug_417();
 }
 
 /** Free all memory held by the routerlist module. */