Browse Source

Re-enable last resort attempt to get via tor.

This looks like a bug introduced in
af658b7828e2ab814d70acbbb99f414dee239def.
Arlo Breault 9 years ago
parent
commit
f752093e16
1 changed files with 4 additions and 5 deletions
  1. 4 5
      src/or/directory.c

+ 4 - 5
src/or/directory.c

@@ -518,14 +518,13 @@ MOCK_IMPL(void, directory_get_from_dirserver, (uint8_t dir_purpose,
         /* */
         rs = directory_pick_generic_dirserver(type, pds_flags,
                                               dir_purpose);
-        if (!rs) {
-          /*XXXX024 I'm pretty sure this can never do any good, since
-           * rs isn't set. */
+        if (!rs)
           get_via_tor = 1; /* last resort: try routing it via Tor */
-        }
       }
     }
-  } else { /* get_via_tor */
+  }
+
+  if (get_via_tor) {
     /* Never use fascistfirewall; we're going via Tor. */
     pds_flags |= PDS_IGNORE_FASCISTFIREWALL;
     rs = router_pick_directory_server(type, pds_flags);