[or-cvs] r14415: Correctly notify one-hop connections when a circuit build ha (in tor/trunk: . src/or)

[nickm at seul.org]

Author: nickm
Date: 2008-04-22 13:44:28 -0400 (Tue, 22 Apr 2008)
New Revision: 14415

Modified:
   tor/trunk/
   tor/trunk/ChangeLog
   tor/trunk/src/or/circuituse.c
Log:
 r15283 at tombo:  nickm | 2008-04-22 13:43:45 -0400
 Correctly notify one-hop connections when a circuit build has failed. possible fix for bug 669. found by lodger.



Property changes on: tor/trunk
___________________________________________________________________
 svk:merge ticket from /tor/trunk [r15283] on 49666b30-7950-49c5-bedf-9dc8f3168102

Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2008-04-22 17:21:26 UTC (rev 14414)
+++ tor/trunk/ChangeLog	2008-04-22 17:44:28 UTC (rev 14415)
@@ -35,6 +35,8 @@
       by adding "new stream" events for DNS requests, and removing
       spurious "stream closed" events" for cached reverse resolves.
       Patch from mwenge.  Fixes bug 646.
+    - Correctly notify one-hop connections when a circuit build has
+      failed.  Possible fix for bug 669.  Found by lodger.
 
   o Minor features:
     - Allow separate log levels to be configured for different logging

Modified: tor/trunk/src/or/circuituse.c
===================================================================
--- tor/trunk/src/or/circuituse.c	2008-04-22 17:21:26 UTC (rev 14414)
+++ tor/trunk/src/or/circuituse.c	2008-04-22 17:44:28 UTC (rev 14415)
@@ -745,10 +745,10 @@
       n_conn->_base.or_is_obsolete = 1;
       entry_guard_register_connect_status(n_conn->identity_digest, 0,
                                           time(NULL));
-      /* if there are any one-hop streams waiting on this circuit, fail
-       * them now so they can retry elsewhere. */
-      connection_ap_fail_onehop(n_conn->identity_digest);
     }
+    /* if there are any one-hop streams waiting on this circuit, fail
+     * them now so they can retry elsewhere. */
+    connection_ap_fail_onehop(circ->_base.n_conn_id_digest);
   }
 
   switch (circ->_base.purpose) {