aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorteor <teor2345@gmail.com>2016-11-07 22:58:36 +1100
committerteor <teor2345@gmail.com>2016-11-07 23:05:55 +1100
commit1bb28cecd92cb72f6df3b7bdf47bdbfa53c5c6dc (patch)
treebefb4e1df5e2e4d657ba1aa3ab5d8bd4b2dabff9
parentf6a3d213e4ace95bbe0361878705d9a889f92974 (diff)
downloadtor-1bb28cecd92cb72f6df3b7bdf47bdbfa53c5c6dc.tar.gz
tor-1bb28cecd92cb72f6df3b7bdf47bdbfa53c5c6dc.zip
Ensure relays don't make multiple connections during bootstrap
Relays do not deliberately launch multiple attempts, so the impact of this bug should be minimal. This fix also defends against bugs like #20499. Bugfix on 0.2.8.1-alpha.
-rw-r--r--changes/bug205913
-rw-r--r--src/or/networkstatus.c2
2 files changed, 4 insertions, 1 deletions
diff --git a/changes/bug20591 b/changes/bug20591
new file mode 100644
index 0000000000..deaa738f5e
--- /dev/null
+++ b/changes/bug20591
@@ -0,0 +1,3 @@
+ o Minor bugfixes (relay bootstrap):
+ - Ensure relays don't make multiple connections during bootstrap.
+ Fixes bug 20591; bugfix on 0.2.8.1-alpha.
diff --git a/src/or/networkstatus.c b/src/or/networkstatus.c
index 49baeb83b7..a6656f5596 100644
--- a/src/or/networkstatus.c
+++ b/src/or/networkstatus.c
@@ -880,7 +880,7 @@ update_consensus_networkstatus_downloads(time_t now)
resource = networkstatus_get_flavor_name(i);
/* Check if we already have enough connections in progress */
- if (we_are_bootstrapping) {
+ if (we_are_bootstrapping && use_multi_conn) {
max_in_progress_conns =
options->ClientBootstrapConsensusMaxInProgressTries;
}