[tor-commits] [tor/maint-0.2.2] put a _ before or_options_t elements that aren't configurable

[arma@xxxxxxxxxxxxxx]

commit 5fed1ccd901d4751a3fed7dc01042771ca76f449
Author: Roger Dingledine <arma@xxxxxxxxxxxxxx>
Date:   Sun Apr 1 15:59:38 2012 -0400

    put a _ before or_options_t elements that aren't configurable
    
    it's fine with me if we change the current convention, but we should
    actually decide to change it if we want to.
---
 src/or/config.c    |    6 +++---
 src/or/directory.c |    5 +++--
 src/or/or.h        |    4 ++--
 3 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/or/config.c b/src/or/config.c
index 1e7bd58..fbbd902 100644
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -713,7 +713,7 @@ or_options_free(or_options_t *options)
     return;
 
   routerset_free(options->_ExcludeExitNodesUnion);
-  tor_free(options->BridgePassword_AuthDigest);
+  tor_free(options->_BridgePassword_AuthDigest);
   config_free(&options_format, options);
 }
 
@@ -1310,8 +1310,8 @@ options_act(or_options_t *old_options)
                "BridgePassword.");
       return -1;
     }
-    options->BridgePassword_AuthDigest = tor_malloc(DIGEST256_LEN);
-    crypto_digest256(options->BridgePassword_AuthDigest,
+    options->_BridgePassword_AuthDigest = tor_malloc(DIGEST256_LEN);
+    crypto_digest256(options->_BridgePassword_AuthDigest,
                      http_authenticator, strlen(http_authenticator),
                      DIGEST_SHA256);
     tor_free(http_authenticator);
diff --git a/src/or/directory.c b/src/or/directory.c
index 9bc58e5..29cf10c 100644
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -3069,7 +3069,7 @@ directory_handle_command_get(dir_connection_t *conn, const char *headers,
   }
 
   if (options->BridgeAuthoritativeDir &&
-      options->BridgePassword_AuthDigest &&
+      options->_BridgePassword_AuthDigest &&
       connection_dir_is_encrypted(conn) &&
       !strcmp(url,"/tor/networkstatus-bridges")) {
     char *status;
@@ -3081,7 +3081,8 @@ directory_handle_command_get(dir_connection_t *conn, const char *headers,
 
     /* now make sure the password is there and right */
     if (!header ||
-        tor_memneq(digest, options->BridgePassword_AuthDigest, DIGEST256_LEN)) {
+        tor_memneq(digest,
+                   options->_BridgePassword_AuthDigest, DIGEST256_LEN)) {
       write_http_status_line(conn, 404, "Not found");
       tor_free(header);
       goto done;
diff --git a/src/or/or.h b/src/or/or.h
index 92592e5..a4e8e49 100644
--- a/src/or/or.h
+++ b/src/or/or.h
@@ -2492,8 +2492,8 @@ typedef struct {
    * for bridge statuses -- but only if the requests use this password. */
   char *BridgePassword;
   /** If BridgePassword is set, this is a SHA256 digest of the basic http
-   * authenticator for it. */
-  char *BridgePassword_AuthDigest;
+   * authenticator for it. Used so we can do a time-independent comparison. */
+  char *_BridgePassword_AuthDigest;
 
   int UseBridges; /**< Boolean: should we start all circuits with a bridge? */
   config_line_t *Bridges; /**< List of bootstrap bridge addresses. */

_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits