diff options
author | Roger Dingledine <arma@torproject.org> | 2016-06-12 16:14:15 -0400 |
---|---|---|
committer | Roger Dingledine <arma@torproject.org> | 2016-06-12 16:14:15 -0400 |
commit | 0616fd6fb6db5385eac6b432892c90fbb38f8666 (patch) | |
tree | df001b9573521a3561d23da5ce20eaf407e8e0e0 /src/or/directory.c | |
parent | 80f2c3555d70261fb62d24f7fb2e9199cf377777 (diff) | |
download | tor-0616fd6fb6db5385eac6b432892c90fbb38f8666.tar.gz tor-0616fd6fb6db5385eac6b432892c90fbb38f8666.zip |
typo/comment/log fixes i found in my sandbox from montreal
Diffstat (limited to 'src/or/directory.c')
-rw-r--r-- | src/or/directory.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/or/directory.c b/src/or/directory.c index abac19db2f..6caca11737 100644 --- a/src/or/directory.c +++ b/src/or/directory.c @@ -727,6 +727,10 @@ directory_initiate_command_routerstatus_rend(const routerstatus_t *status, node = node_get_by_id(status->identity_digest); + /* XXX The below check is wrong: !node means it's not in the consensus, + * but we haven't checked if we have a descriptor for it -- and also, + * we only care about the descriptor if it's a begindir-style anonymized + * connection. */ if (!node && anonymized_connection) { log_info(LD_DIR, "Not sending anonymized request to directory '%s'; we " "don't have its router descriptor.", @@ -744,7 +748,7 @@ directory_initiate_command_routerstatus_rend(const routerstatus_t *status, return; } - /* At this point, if we are a clients making a direct connection to a + /* At this point, if we are a client making a direct connection to a * directory server, we have selected a server that has at least one address * allowed by ClientUseIPv4/6 and Reachable{"",OR,Dir}Addresses. This * selection uses the preference in ClientPreferIPv6{OR,Dir}Port, if |