if connecting to an OR fails immediately, mark it as down

svn:r2029

diff --git a/src/or/connection.c b/src/or/connection.c
index 90c8197c46aa5c84a67e98d1ed83c1e23e96f3bb..578466baed9e6f4c5722434c3ec70e121915fafb 100644 (file)
--- a/src/or/connection.c
+++ b/src/or/connection.c
@@ -829,7 +829,9 @@ int connection_handle_write(connection_t *conn) {
         log_fn(LOG_DEBUG,"in-progress connect failed. Removing.");
         connection_close_immediate(conn);
         connection_mark_for_close(conn);
-        /* Previously we tested conn->nickname; is this right? */
+        /* it's safe to pass OPs to router_mark_as_down(), since it just
+         * ignores unrecognized routers
+         */
         if (conn->type == CONN_TYPE_OR)
           router_mark_as_down(conn->identity_digest);
         return -1;

diff --git a/src/or/connection_or.c b/src/or/connection_or.c
index 667a82d92ba2a1e4c4bf10167a0a4f83e6e506c1..87e2f9df2a622fc96306f40f63e8bac451b76251 100644 (file)
--- a/src/or/connection_or.c
+++ b/src/or/connection_or.c
@@ -177,6 +177,7 @@ connection_t *connection_or_connect(uint32_t addr, uint16_t port,
 
   switch(connection_connect(conn, conn->address, addr, port)) {
     case -1:
+      router_mark_as_down(conn->identity_digest);
       connection_free(conn);
       return NULL;
     case 0:

diff --git a/src/or/routerlist.c b/src/or/routerlist.c
index 1bd21e44115d8152a3fcb1aff48e205e32fb95e6..7874b1075567f3806b5a0dfa486152f1f5ff787c 100644 (file)
--- a/src/or/routerlist.c
+++ b/src/or/routerlist.c
@@ -354,7 +354,7 @@ void routerlist_free(routerlist_t *rl)
   tor_free(rl);
 }
 
-/** Mark the router named <b>nickname</b> as non-running in our routerlist. */
+/** Mark the router with ID <b>digest</b> as non-running in our routerlist. */
 void router_mark_as_down(const char *digest) {
   routerinfo_t *router;
   tor_assert(digest);