[tor-commits] [tor/master] fix typos/etc before i go nuts on #18809

arma at torproject.org arma at torproject.org
Wed Apr 13 04:07:00 UTC 2016


commit 525307c0ea5717adba9f35cf7ccb1c5108ac3440
Author: Roger Dingledine <arma at torproject.org>
Date:   Wed Apr 13 00:06:30 2016 -0400

    fix typos/etc before i go nuts on #18809
---
 src/or/directory.c         | 12 ++++++------
 src/or/main.c              |  2 +-
 src/or/networkstatus.c     | 17 +++++++++--------
 src/or/networkstatus.h     |  2 +-
 src/test/test_connection.c |  2 +-
 5 files changed, 18 insertions(+), 17 deletions(-)

diff --git a/src/or/directory.c b/src/or/directory.c
index dbb2987..45061ae 100644
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -3648,7 +3648,7 @@ connection_dir_would_close_consensus_conn_helper(void)
    * consensus, and we are still bootstrapping (that is, we have no usable
    * consensus), we don't want to close any until one starts downloading. */
   if (!networkstatus_consensus_is_downloading_usable_flavor()
-      && networkstatus_consensus_is_boostrapping(time(NULL))) {
+      && networkstatus_consensus_is_bootstrapping(time(NULL))) {
     return 0;
   }
 
@@ -3682,7 +3682,7 @@ connection_dir_avoid_extra_connection_for_purpose(unsigned int purpose)
    * bootstrapping (that is, we have no usable consensus), we can be sure that
    * any further connections would be excess. */
   if (networkstatus_consensus_is_downloading_usable_flavor()
-      && networkstatus_consensus_is_boostrapping(time(NULL))) {
+      && networkstatus_consensus_is_bootstrapping(time(NULL))) {
     return 1;
   }
 
@@ -3723,12 +3723,12 @@ connection_dir_close_consensus_conn_if_extra(dir_connection_t *conn)
     return 0;
   }
 
-  const int we_are_bootstrapping = networkstatus_consensus_is_boostrapping(
+  const int we_are_bootstrapping = networkstatus_consensus_is_bootstrapping(
                                                                   time(NULL));
 
   /* We don't want to check other connections to see if they are downloading,
    * as this is prone to race-conditions. So leave it for
-   * connection_dir_consider_close_extra_consensus_conns() to clean up.
+   * connection_dir_close_extra_consensus_conns(() to clean up.
    *
    * But if conn has just started connecting, or we have a consensus already,
    * we can be sure it's not needed any more. */
@@ -3768,7 +3768,7 @@ connection_dir_close_extra_consensus_conns(void)
     return;
   }
 
-  int we_are_bootstrapping = networkstatus_consensus_is_boostrapping(
+  int we_are_bootstrapping = networkstatus_consensus_is_bootstrapping(
                                                                   time(NULL));
 
   const char *usable_resource = networkstatus_get_flavor_name(
@@ -3877,7 +3877,7 @@ find_dl_schedule(download_status_t *dls, const or_options_t *options)
   const int dir_server = dir_server_mode(options);
   const int multi_d = networkstatus_consensus_can_use_multiple_directories(
                                                                     options);
-  const int we_are_bootstrapping = networkstatus_consensus_is_boostrapping(
+  const int we_are_bootstrapping = networkstatus_consensus_is_bootstrapping(
                                                                  time(NULL));
   const int use_fallbacks = networkstatus_consensus_can_use_extra_fallbacks(
                                                                     options);
diff --git a/src/or/main.c b/src/or/main.c
index a2cf5b1..bf5b2b8 100644
--- a/src/or/main.c
+++ b/src/or/main.c
@@ -1917,7 +1917,7 @@ fetch_networkstatus_callback(time_t now, const or_options_t *options)
 
   /* How often do we check whether we should download network status
    * documents? */
-  const int we_are_bootstrapping = networkstatus_consensus_is_boostrapping(
+  const int we_are_bootstrapping = networkstatus_consensus_is_bootstrapping(
                                                                         now);
   const int prefer_mirrors = !directory_fetches_from_authorities(
                                                               get_options());
diff --git a/src/or/networkstatus.c b/src/or/networkstatus.c
index 3967f56..51c985e 100644
--- a/src/or/networkstatus.c
+++ b/src/or/networkstatus.c
@@ -819,7 +819,7 @@ update_consensus_networkstatus_downloads(time_t now)
 {
   int i;
   const or_options_t *options = get_options();
-  const int we_are_bootstrapping = networkstatus_consensus_is_boostrapping(
+  const int we_are_bootstrapping = networkstatus_consensus_is_bootstrapping(
                                                                         now);
   const int use_multi_conn =
     networkstatus_consensus_can_use_multiple_directories(options);
@@ -875,12 +875,13 @@ update_consensus_networkstatus_downloads(time_t now)
                                                    resource,
                                                    DIR_CONN_STATE_CONNECTING);
 
-      if (i == usable_consensus_flavor()
-          && connect_consens_conn_count < consens_conn_count) {
+      /* If not all connections are "connecting", then some are
+       * downloading. We want to have at most one downloading at a time. */
+      if (connect_consens_conn_count < consens_conn_count) {
         continue;
       }
 
-      /* Make multiple connections for a bootstrap consensus download */
+      /* Make multiple connections for a bootstrap consensus download. */
       update_consensus_bootstrap_multiple_downloads(now, options,
                                                     we_are_bootstrapping);
     } else {
@@ -954,7 +955,7 @@ update_consensus_bootstrap_attempt_downloads(
  * connections.
  * Only call when bootstrapping, and when we want to make additional
  * connections. Only nodes that satisfy
- * networkstatus_consensus_can_use_multiple_directories make additonal
+ * networkstatus_consensus_can_use_multiple_directories make additional
  * connections.
  */
 static void
@@ -969,7 +970,7 @@ update_consensus_bootstrap_multiple_downloads(time_t now,
     return;
   }
 
-  /* If we've managed to validate a usable consensus, don't make additonal
+  /* If we've managed to validate a usable consensus, don't make additional
    * connections. */
   if (!we_are_bootstrapping) {
     return;
@@ -1277,7 +1278,7 @@ networkstatus_get_reasonably_live_consensus(time_t now, int flavor)
  *  only using the authorities and fallback directory mirrors to download the
  * consensus flavour we'll use. */
 int
-networkstatus_consensus_is_boostrapping(time_t now)
+networkstatus_consensus_is_bootstrapping(time_t now)
 {
   /* If we don't have a consensus, we must still be bootstrapping */
   return !networkstatus_get_reasonably_live_consensus(
@@ -1327,7 +1328,7 @@ networkstatus_consensus_can_use_extra_fallbacks(const or_options_t *options)
  * return value of this function to see if a client could make multiple
  * bootstrap connections. Use
  * networkstatus_consensus_can_use_multiple_directories()
- * and networkstatus_consensus_is_boostrapping(). */
+ * and networkstatus_consensus_is_bootstrapping(). */
 int
 networkstatus_consensus_has_excess_connections(void)
 {
diff --git a/src/or/networkstatus.h b/src/or/networkstatus.h
index 9bbb9a3..f2f8af5 100644
--- a/src/or/networkstatus.h
+++ b/src/or/networkstatus.h
@@ -70,7 +70,7 @@ MOCK_DECL(networkstatus_t *,networkstatus_get_latest_consensus_by_flavor,
 networkstatus_t *networkstatus_get_live_consensus(time_t now);
 networkstatus_t *networkstatus_get_reasonably_live_consensus(time_t now,
                                                              int flavor);
-int networkstatus_consensus_is_boostrapping(time_t now);
+int networkstatus_consensus_is_bootstrapping(time_t now);
 int networkstatus_consensus_can_use_multiple_directories(
                                                 const or_options_t *options);
 int networkstatus_consensus_can_use_extra_fallbacks(
diff --git a/src/test/test_connection.c b/src/test/test_connection.c
index 15ae973..6f7aef8 100644
--- a/src/test/test_connection.c
+++ b/src/test/test_connection.c
@@ -705,7 +705,7 @@ test_conn_download_status(void *arg)
 
   /* now try closing the one that isn't downloading:
    * these tests won't work unless tor thinks it is bootstrapping */
-  tt_assert(networkstatus_consensus_is_boostrapping(time(NULL)));
+  tt_assert(networkstatus_consensus_is_bootstrapping(time(NULL)));
 
   tt_assert(connection_dir_count_by_purpose_and_resource(
                                                         TEST_CONN_RSRC_PURPOSE,



More information about the tor-commits mailing list