Browse Source

r16665@catbus: nickm | 2007-11-14 13:36:11 -0500
Fix bug spotted by karsten: when we have no consensus, do not try to continue finding directories for a hidden service.


svn:r12493

Nick Mathewson 16 years ago
parent
commit
bbb34846b4
2 changed files with 6 additions and 0 deletions
  1. 5 0
      ChangeLog
  2. 1 0
      src/or/routerlist.c

+ 5 - 0
ChangeLog

@@ -3,6 +3,11 @@ Changes in version 0.2.0.12-alpha - 2007-11-??
     - Make it build on OpenBSD again. Patch from tup.
     - Substitute BINDIR and LOCALSTATEDIR in scripts.  Fixes packaging.
 
+  o Minor bugfixes:
+    - When we're lacking a consensus, don't try to perform rendezvous
+      operations.  (Bug spotted by Karsten)
+
+
 Changes in version 0.2.0.11-alpha - 2007-11-12
   o Security fixes:
     - Exit policies now reject connections that are addressed to a

+ 1 - 0
src/or/routerlist.c

@@ -4472,6 +4472,7 @@ hid_serv_get_responsible_directories(smartlist_t *responsible_dirs,
   if (!c || !smartlist_len(c->routerstatus_list)) {
     log_warn(LD_REND, "We don't have a consensus, so we can't perform v2 "
              "rendezvous operations.");
+    return -1;
   }
   tor_assert(id);
   start = networkstatus_vote_find_entry_idx(c, id, &found);