[tor-commits] [tor/master] Ensure relays don't make multiple connections during bootstrap

[nickm at torproject.org]

commit 1bb28cecd92cb72f6df3b7bdf47bdbfa53c5c6dc
Author: teor <teor2345 at gmail.com>
Date:   Mon Nov 7 22:58:36 2016 +1100

    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.
---
 changes/bug20591       | 3 +++
 src/or/networkstatus.c | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/changes/bug20591 b/changes/bug20591
new file mode 100644
index 0000000..deaa738
--- /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 49baeb8..a6656f5 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;
     }