[tor-commits] [tor/maint-0.2.7] Fix: use the right list in find_expiring_intro_point()

nickm at torproject.org nickm at torproject.org
Wed Nov 25 13:53:30 UTC 2015


commit 273b267fa234c5a26c9beaacede42a496c880e05
Author: David Goulet <dgoulet at ev0ke.net>
Date:   Mon Nov 23 09:02:54 2015 -0500

    Fix: use the right list in find_expiring_intro_point()
    
    The wrong list was used when looking up expired intro points in a rend
    service object causing what we think could be reachability issues and
    triggering a BUG log.
    
    Fixes #16702
    
    Signed-off-by: David Goulet <dgoulet at ev0ke.net>
---
 changes/bug16702     |    4 ++++
 src/or/rendservice.c |    3 ++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/changes/bug16702 b/changes/bug16702
new file mode 100644
index 0000000..5de36cd
--- /dev/null
+++ b/changes/bug16702
@@ -0,0 +1,4 @@
+  o Minor bugfixes (hidden service)
+    - The wrong list was used when looking up expired intro points in a rend
+      service object causing what we think could be reachability issues and
+      triggering a BUG log. Fixes 16702; bugfix on tor-0.2.7.2-alpha.
diff --git a/src/or/rendservice.c b/src/or/rendservice.c
index 8ba5327..77d8b71 100644
--- a/src/or/rendservice.c
+++ b/src/or/rendservice.c
@@ -3038,7 +3038,8 @@ find_expiring_intro_point(rend_service_t *service, origin_circuit_t *circ)
   tor_assert(TO_CIRCUIT(circ)->purpose == CIRCUIT_PURPOSE_S_ESTABLISH_INTRO ||
              TO_CIRCUIT(circ)->purpose == CIRCUIT_PURPOSE_S_INTRO);
 
-  SMARTLIST_FOREACH(service->intro_nodes, rend_intro_point_t *, intro_point,
+  SMARTLIST_FOREACH(service->expiring_nodes, rend_intro_point_t *,
+                    intro_point,
     if (crypto_pk_eq_keys(intro_point->intro_key, circ->intro_key)) {
       return intro_point;
   });



More information about the tor-commits mailing list