bugfix on r9568: we were throwing around an uninitialized

nickname array, and only treating a server as reachable if
the array magically has the right nickname in it. this was
causing the authorities to label only themselves as running,
and clients were "mysteriously" failing.

svn:r9628

diff --git a/src/or/connection_or.c b/src/or/connection_or.c
index bd1554be5bfe999b93a6986dce2eca25f6b30972..34c890b5ae07a61e6b98b5bdcebacee6dc41658c 100644 (file)
--- a/src/or/connection_or.c
+++ b/src/or/connection_or.c
@@ -591,7 +591,6 @@ connection_or_check_valid_handshake(or_connection_t *conn, int started_here,
                                     char *digest_rcvd)
 {
   crypto_pk_env_t *identity_rcvd=NULL;
-  char nickname[MAX_NICKNAME_LEN+1];
   or_options_t *options = get_options();
   int severity = server_mode(options) ? LOG_PROTOCOL_WARN : LOG_WARN;
   const char *safe_address =
@@ -668,10 +667,10 @@ connection_or_check_valid_handshake(or_connection_t *conn, int started_here,
     }
     if (authdir_mode(options)) {
       /* We initiated this connection to address:port.  Drop all routers
-       * with the same address:port and a different key or nickname.
+       * with the same address:port and a different key.
        */
       dirserv_orconn_tls_done(conn->_base.address, conn->_base.port,
-                              digest_rcvd, nickname, as_advertised);
+                              digest_rcvd, as_advertised);
     }
     if (!as_advertised)
       return -1;

diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index 07ebe771779abd97924d20368613ca14c54eed5e..d2b6661935488cf5050da94645bbab4b8d97c575 100644 (file)
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -1914,8 +1914,7 @@ dirserv_get_routerdescs(smartlist_t *descs_out, const char *key,
 
 /** Called when a TLS handshake has completed successfully with a
  * router listening at <b>address</b>:<b>or_port</b>, and has yielded
- * a certificate with digest <b>digest_rcvd</b> and nickname
- * <b>nickname_rcvd</b>.
+ * a certificate with digest <b>digest_rcvd</b>.
  *
  * Also, if as_advertised is 1, then inform the reachability checker
  * that we could get to this guy.
@@ -1924,20 +1923,17 @@ void
 dirserv_orconn_tls_done(const char *address,
                         uint16_t or_port,
                         const char *digest_rcvd,
-                        const char *nickname_rcvd,
                         int as_advertised)
 {
   routerlist_t *rl = router_get_routerlist();
   tor_assert(address);
   tor_assert(digest_rcvd);
-  tor_assert(nickname_rcvd);
 
   SMARTLIST_FOREACH(rl->routers, routerinfo_t *, ri, {
     if (!strcasecmp(address, ri->address) && or_port == ri->or_port &&
         as_advertised &&
-        !memcmp(ri->cache_info.identity_digest, digest_rcvd, DIGEST_LEN) &&
-        !strcasecmp(nickname_rcvd, ri->nickname)) {
-      /* correct nickname and digest. mark this router reachable! */
+        !memcmp(ri->cache_info.identity_digest, digest_rcvd, DIGEST_LEN)) {
+      /* correct digest. mark this router reachable! */
       log_info(LD_DIRSERV, "Found router %s to be reachable. Yay.",
                ri->nickname);
       ri->last_reachable = time(NULL);

diff --git a/src/or/or.h b/src/or/or.h
index 1eb8702f324a573ad8719865a2cf53f50ecd612a..54b2f15bcd7952646ae0be7613bf8e739e6c4354 100644 (file)
--- a/src/or/or.h
+++ b/src/or/or.h
@@ -2465,7 +2465,6 @@ int dirserv_get_routerdescs(smartlist_t *descs_out, const char *key,
 void dirserv_orconn_tls_done(const char *address,
                              uint16_t or_port,
                              const char *digest_rcvd,
-                             const char *nickname,
                              int as_advertised);
 void dirserv_test_reachability(int try_all);
 int authdir_wants_to_reject_router(routerinfo_t *ri, const char **msg,