commit e0ea7407a4370c977ebbf0b70712c9e5ff7937fa
parent 0e00d9369fa159bf0c44d0862e3d1f6b0c40ef50
Author: teor <teor@torproject.org>
Date: Wed, 12 Feb 2020 14:14:11 +1000
Merge remote-tracking branch 'tor-github/pr/1725' into maint-0.4.3
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
@@ -633,9 +633,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;
}
