From 88489cd29006f093e1a5572711b26ee1b057e359 Mon Sep 17 00:00:00 2001 From: David Goulet Date: Tue, 4 Feb 2020 10:13:10 -0500 Subject: hs-v3: Remove BUG() that can occur normally Fixes #28992 Signed-off-by: David Goulet --- src/feature/hs/hs_client.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'src/feature') diff --git a/src/feature/hs/hs_client.c b/src/feature/hs/hs_client.c index fd2d266453..b045e1899b 100644 --- a/src/feature/hs/hs_client.c +++ b/src/feature/hs/hs_client.c @@ -596,9 +596,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; } -- cgit v1.2.3-54-g00ecf