Browse Source

stop looping once we know what the answer will be

suggested during code review by dgoulet
Roger Dingledine 8 years ago
parent
commit
aa6341d4b9
1 changed files with 3 additions and 1 deletions
  1. 3 1
      src/or/networkstatus.c

+ 3 - 1
src/or/networkstatus.c

@@ -1347,8 +1347,10 @@ networkstatus_consensus_is_already_downloading(const char *resource)
     connection_t *base = TO_CONN(dirconn);
     if (base->linked_conn &&
         base->linked_conn->type == CONN_TYPE_AP &&
-        !AP_CONN_STATE_IS_UNATTACHED(base->linked_conn->state))
+        !AP_CONN_STATE_IS_UNATTACHED(base->linked_conn->state)) {
       answer = 1;
+      break; /* stop looping, because we know the answer will be yes */
+    }
   } SMARTLIST_FOREACH_END(dirconn);
   smartlist_free(fetching_conns);