[or-cvs] r16810: {tor} minor cleanups on karsten's patch (in tor/trunk: . src/or)

[arma at seul.org]

Author: arma
Date: 2008-09-09 05:21:21 -0400 (Tue, 09 Sep 2008)
New Revision: 16810

Modified:
   tor/trunk/ChangeLog
   tor/trunk/src/or/main.c
   tor/trunk/src/or/rendservice.c
Log:
minor cleanups on karsten's patch


Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2008-09-09 09:14:16 UTC (rev 16809)
+++ tor/trunk/ChangeLog	2008-09-09 09:21:21 UTC (rev 16810)
@@ -5,12 +5,13 @@
       relay. Fixes bug 809. Bugfix on 0.2.1.5-alpha.
     - When extending a circuit to a hidden service directory to upload a
       rendezvous descriptor using a BEGIN_DIR cell, almost 1/6 of all
-      requests failed, because the router descriptor has not been downloaded
-      before. In these cases, do not attempt to upload the rendezvous
-      descriptor, but wait until the router descriptor is downloaded and
-      retry. Likewise, do not attempt to fetch a rendezvous from a hidden
-      service directory for which the router descriptor has not been
-      downloaded, yet. Fixes bug 767. Bugfix on 0.2.0.10-alpha.
+      requests failed, because the router descriptor has not been
+      downloaded yet. In these cases, do not attempt to upload the
+      rendezvous descriptor, but wait until the router descriptor is
+      downloaded and retry. Likewise, do not attempt to fetch a rendezvous
+      descriptor from a hidden service directory for which the router
+      descriptor has not yet been downloaded. Fixes bug 767. Bugfix
+      on 0.2.0.10-alpha.
 
   o Minor bugfixes:
     - Fix compile on OpenBSD 4.4-current. Bugfix on 0.2.1.5-alpha.

Modified: tor/trunk/src/or/main.c
===================================================================
--- tor/trunk/src/or/main.c	2008-09-09 09:14:16 UTC (rev 16809)
+++ tor/trunk/src/or/main.c	2008-09-09 09:21:21 UTC (rev 16810)
@@ -1127,8 +1127,8 @@
 
   /** 7. And upload service descriptors if necessary. */
   if (has_completed_circuit && !we_are_hibernating()) {
+    rend_consider_services_upload(now);
     rend_consider_descriptor_republication();
-    rend_consider_services_upload(now);
   }
 
   /** 8. and blow away any connections that need to die. have to do this now,

Modified: tor/trunk/src/or/rendservice.c
===================================================================
--- tor/trunk/src/or/rendservice.c	2008-09-09 09:14:16 UTC (rev 16809)
+++ tor/trunk/src/or/rendservice.c	2008-09-09 09:21:21 UTC (rev 16810)
@@ -1420,7 +1420,7 @@
       SMARTLIST_FOREACH(renddesc->successful_uploads, char *, c, tor_free(c););
       smartlist_free(renddesc->successful_uploads);
     }
-    renddesc->all_uploads_performed = -1;
+    renddesc->all_uploads_performed = 1;
   } else {
     /* Remember which routers worked this time, so that we don't upload the
      * descriptor to them again. */