From 588c77677a84a4c7b7222bafd6fb6ad80826064a Mon Sep 17 00:00:00 2001 From: teor Date: Fri, 24 Aug 2018 11:55:41 +1000 Subject: [PATCH] Bootstrap: stop requiring descriptors to count exits as usable Instead, count exits as usable if they have the exit flag, and present if they also have a non-reject exit policy. Requiring a threshold of usable descriptors avoids directories trickling exit descriptors to clients to discover their ExitNodes settings. Part of 27236. --- src/or/nodelist.c | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/src/or/nodelist.c b/src/or/nodelist.c index 85e4ae38d3..07632861d1 100644 --- a/src/or/nodelist.c +++ b/src/or/nodelist.c @@ -2118,9 +2118,9 @@ get_dir_info_status_string(void) * * If in_set is non-NULL, only consider those routers in in_set. * If exit_only & USABLE_DESCRIPTOR_EXIT_POLICY, only consider nodes - * with an exit policy that accepts at least one port. + * present if they have an exit policy that accepts at least one port. * If exit_only & USABLE_DESCRIPTOR_EXIT_FLAG, only consider nodes - * with the exit flag in the consensus. + * usable if they have the exit flag in the consensus. * * If *descs_out is present, add a node_t for each usable descriptor * to it. @@ -2147,12 +2147,6 @@ count_usable_descriptors(int *num_present, int *num_usable, if (in_set && ! routerset_contains_routerstatus(in_set, rs, -1)) continue; if (client_would_use_router(rs, now)) { - /* Do the policy check last, because it's potentially expensive */ - if ((exit_only & USABLE_DESCRIPTOR_EXIT_POLICY) && - node_has_preferred_descriptor(node, 0) && - node_exit_policy_rejects_all(node)) { - continue; - } const char * const digest = rs->descriptor_digest; int present; ++*num_usable; /* the consensus says we want it. */ @@ -2161,7 +2155,14 @@ count_usable_descriptors(int *num_present, int *num_usable, else present = NULL != router_get_by_descriptor_digest(digest); if (present) { - /* we have the descriptor listed in the consensus. */ + /* Do the policy check last, because it requires a descriptor, + * and is potentially expensive */ + if ((exit_only & USABLE_DESCRIPTOR_EXIT_POLICY) && + node_exit_policy_rejects_all(node)) { + continue; + } + /* we have the descriptor listed in the consensus, and it + * satisfies our exit constraints (if any) */ ++*num_present; } if (descs_out) @@ -2239,7 +2240,7 @@ compute_frac_paths_available(const networkstatus_t *consensus, np, nu); - /* We need at least 1 exit present in the consensus to consider + /* We need at least 1 exit usable in the consensus to consider * building exit paths */ /* Update our understanding of whether the consensus has exits */ consensus_path_type_t old_have_consensus_path = have_consensus_path;