[tor-commits] [tor/master] Actually, both nacks and acks indicate a valid path

[nickm@xxxxxxxxxxxxxx]

commit c3b71a3fc96c6f3eaaebd96ef8c15d4298d9639e
Author: Mike Perry <mikeperry-git@xxxxxxxxxx>
Date:   Fri Dec 7 15:50:31 2012 -0800

    Actually, both nacks and acks indicate a valid path
---
 src/or/rendclient.c |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/or/rendclient.c b/src/or/rendclient.c
index ec43041..1d473de 100644
--- a/src/or/rendclient.c
+++ b/src/or/rendclient.c
@@ -361,6 +361,10 @@ rend_client_introduction_acked(origin_circuit_t *circ,
 #endif
   tor_assert(circ->rend_data);
 
+  /* For path bias: This circuit was used successfully. Valid
+   * nacks and acks count. */
+  circ->any_streams_succeeded = 1;
+
   if (request_len == 0) {
     /* It's an ACK; the introduction point relayed our introduction request. */
     /* Locate the rend circ which is waiting to hear about this ack,
@@ -378,9 +382,6 @@ rend_client_introduction_acked(origin_circuit_t *circ,
        * it to specify when a circuit entered the
        * _C_REND_READY_INTRO_ACKED state. */
       rendcirc->base_.timestamp_dirty = time(NULL);
-      
-      /* For path bias: This circuit was used successfully */
-      circ->any_streams_succeeded = 1;
     } else {
       log_info(LD_REND,"...Found no rend circ. Dropping on the floor.");
     }



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