commit fd8ce812b0a7263b75096207a735c02945320c86
parent 0f218a05a6d8271639a81ec2667f86ef1aa38e66
Author: Nick Mathewson <nickm@torproject.org>
Date: Mon, 29 Jun 2020 12:55:35 -0400
Merge branch 'maint-0.3.5' into maint-0.4.2
Diffstat:
2 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/changes/ticket28992 b/changes/ticket28992
@@ -0,0 +1,3 @@
+ o Minor bugfixes (onion service v3, client):
+ - Remove a BUG() that is causing a stacktrace for a situation that very
+ rarely happens but still can. Fixes bug 28992; bugfix on 0.3.2.1-alpha.
diff --git a/src/feature/hs/hs_client.c b/src/feature/hs/hs_client.c
@@ -606,9 +606,14 @@ send_introduce1(origin_circuit_t *intro_circ,
/* We need to find which intro point in the descriptor we are connected to
* on intro_circ. */
ip = find_desc_intro_point_by_ident(intro_circ->hs_ident, desc);
- if (BUG(ip == NULL)) {
- /* If we can find a descriptor from this introduction circuit ident, we
- * must have a valid intro point object. Permanent error. */
+ if (ip == NULL) {
+ /* The following is possible if the descriptor was changed while we had
+ * this introduction circuit open and waiting for the rendezvous circuit to
+ * be ready. Which results in this situation where we can't find the
+ * corresponding intro point within the descriptor of the service. */
+ log_info(LD_REND, "Unable to find introduction point for service %s "
+ "while trying to send an INTRODUCE1 cell.",
+ safe_str_client(onion_address));
goto perm_err;
}
