diff options
author | Nick Mathewson <nickm@torproject.org> | 2017-06-19 15:48:47 -0400 |
---|---|---|
committer | Nick Mathewson <nickm@torproject.org> | 2017-06-19 15:48:47 -0400 |
commit | 32e486de97be6f0267c1318298808488baf6e319 (patch) | |
tree | e5d0a0d63f048d676d9e23f8909824a7c47a4e46 /src/or/entrynodes.c | |
parent | 71c701927a27723f0bc9009a044a4bdece55f361 (diff) | |
download | tor-32e486de97be6f0267c1318298808488baf6e319.tar.gz tor-32e486de97be6f0267c1318298808488baf6e319.zip |
Don't expand guard sample set unless consensus is "reasonably live"
Fixes what I think is the main root cause of 22400. Bugfix on
0.3.0.1-alpha.
Diffstat (limited to 'src/or/entrynodes.c')
-rw-r--r-- | src/or/entrynodes.c | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/src/or/entrynodes.c b/src/or/entrynodes.c index 5b5e13bced..34dfdcef15 100644 --- a/src/or/entrynodes.c +++ b/src/or/entrynodes.c @@ -1104,6 +1104,21 @@ entry_guards_expand_sample(guard_selection_t *gs) { tor_assert(gs); const or_options_t *options = get_options(); + + if (gs->type != GS_TYPE_BRIDGE) { + networkstatus_t *consensus = networkstatus_get_latest_consensus(); + time_t now = approx_time(); + if (consensus == NULL) { + log_info(LD_GUARD, "Not expanding the sample guard set; we have " + "no consensus."); + return NULL; + } else if (!networkstatus_consensus_reasonably_live(consensus, now)) { + log_info(LD_GUARD, "Not expanding the sample guard set; we have " + "a consensus, but it is far too old."); + return NULL; + } + } + int n_sampled = smartlist_len(gs->sampled_entry_guards); entry_guard_t *added_guard = NULL; int n_usable_filtered_guards = num_reachable_filtered_guards(gs, NULL); |