commit 6300a9548dcbe11935db374c7e7455743f3283c3 parent 28413e75605cc2d05a2a3e4c766bfbe0a47d848d Author: David Goulet <dgoulet@torproject.org> Date: Mon, 24 Oct 2022 11:03:38 -0400 hs: Retry rdv circuit if repurposed This can happen if our measurement subsystem decides to snatch it. Fixes #40696 Signed-off-by: David Goulet <dgoulet@torproject.org> Diffstat:
| A | changes/ticket40696 | | | 3 | +++ |
| M | src/feature/hs/hs_circuit.c | | | 11 | +++++++++++ |
2 files changed, 14 insertions(+), 0 deletions(-)
diff --git a/changes/ticket40696 b/changes/ticket40696 @@ -0,0 +1,3 @@ + o Minor bugfixes (onion service): + - Make the service retry a rendezvous if the circuit is being repurposed for + measurements. Fixes bug 40696; bugfix on 0.3.5.1-alpha. diff --git a/src/feature/hs/hs_circuit.c b/src/feature/hs/hs_circuit.c @@ -1319,6 +1319,17 @@ hs_circ_cleanup_on_repurpose(circuit_t *circ) if (circ->hs_token) { hs_circuitmap_remove_circuit(circ); } + + switch (circ->purpose) { + case CIRCUIT_PURPOSE_S_CONNECT_REND: + /* This circuit was connecting to a rendezvous point but it is being + * repurposed so we need to relaunch an attempt else the client will be + * left hanging waiting for the rendezvous. */ + hs_circ_retry_service_rendezvous_point(TO_ORIGIN_CIRCUIT(circ)); + break; + default: + break; + } } /** Return true iff the given established client rendezvous circuit was sent