[or-cvs] r15119: When we're checking if we have enough dir info for each rela (in tor/branches/tor-0_2_0-patches: . doc src/or)

[arma@xxxxxxxx]

Author: arma
Date: 2008-06-10 23:21:03 -0400 (Tue, 10 Jun 2008)
New Revision: 15119

Modified:
   tor/branches/tor-0_2_0-patches/ChangeLog
   tor/branches/tor-0_2_0-patches/doc/TODO.020
   tor/branches/tor-0_2_0-patches/src/or/routerlist.c
Log:
When we're checking if we have enough dir info for each relay
to begin establishing circuits, make sure that we actually have
the descriptor listed in the consensus, not just any descriptor.


Modified: tor/branches/tor-0_2_0-patches/ChangeLog
===================================================================
--- tor/branches/tor-0_2_0-patches/ChangeLog	2008-06-11 02:04:59 UTC (rev 15118)
+++ tor/branches/tor-0_2_0-patches/ChangeLog	2008-06-11 03:21:03 UTC (rev 15119)
@@ -4,11 +4,14 @@
       put in our relay end cell that we send to the exit relay, Tor
       clients on Windows were sometimes sending the wrong 'reason'. The
       anonymity problem is that exit relays may be able to guess whether
-      client is running Windows, thus helping partition the anonymity
+      the client is running Windows, thus helping partition the anonymity
       set. Down the road we should stop sending reasons to exit relays,
       or otherwise prevent future versions of this bug.
 
   o Minor fixes:
+    - When we're checking if we have enough dir info for each relay
+      to begin establishing circuits, make sure that we actually have
+      the descriptor listed in the consensus, not just any descriptor.
     - Bridge relays no longer print "xx=0" in their extrainfo document
       for every single country code in the geoip db.
     - Only warn when we fail to load the geoip file if we were planning to

Modified: tor/branches/tor-0_2_0-patches/doc/TODO.020
===================================================================
--- tor/branches/tor-0_2_0-patches/doc/TODO.020	2008-06-11 02:04:59 UTC (rev 15118)
+++ tor/branches/tor-0_2_0-patches/doc/TODO.020	2008-06-11 03:21:03 UTC (rev 15119)
@@ -4,7 +4,7 @@
 
 Items before a stable 0.2.0.x bundle:
   - Tor items:
-R   - backport r14970
+    X backport r14970
 R   - releasenotes
   - Vidalia items:
 E   d address arma's Linux 100% cpu bug
@@ -21,6 +21,6 @@
 
 Backport for 0.2.0 once better tested:
   - r14830: disable openssl compression.
-  - r14971: we don't have enough dir info if we have mostly wrong
+  o r14971: we don't have enough dir info if we have mostly wrong
     descriptors
 

Modified: tor/branches/tor-0_2_0-patches/src/or/routerlist.c
===================================================================
--- tor/branches/tor-0_2_0-patches/src/or/routerlist.c	2008-06-11 02:04:59 UTC (rev 15118)
+++ tor/branches/tor-0_2_0-patches/src/or/routerlist.c	2008-06-11 03:21:03 UTC (rev 15119)
@@ -4162,7 +4162,7 @@
      {
        if (client_would_use_router(rs, now, options)) {
          ++num_usable;
-         if (router_get_by_digest(rs->identity_digest)) {
+         if (router_get_by_descriptor_digest(rs->descriptor_digest)) {
            ++num_present;
          }
        }