diff options
| author | George Kadianakis <desnacked@riseup.net> | 2020-07-09 17:08:34 +0300 |
|---|---|---|
| committer | George Kadianakis <desnacked@riseup.net> | 2020-07-09 17:08:34 +0300 |
| commit | b1a32761bec22443b8ac745d798b86a4c9d5bd2d (patch) | |
| tree | d2ebca2ba1b5266eaa77550f5efa00322184629e | |
| parent | be6b30e642eafc8ad9552e08ab117ac2a9900623 (diff) | |
| parent | e500ea2eae0652dad94e16cf2765044146740ce3 (diff) | |
| download | tor-b1a32761bec22443b8ac745d798b86a4c9d5bd2d.tar.gz tor-b1a32761bec22443b8ac745d798b86a4c9d5bd2d.zip | |
Merge branch 'maint-0.4.4'
| -rw-r--r-- | changes/ticket34087 | 3 | ||||
| -rw-r--r-- | src/feature/hs/hs_client.c | 6 |
2 files changed, 7 insertions, 2 deletions
diff --git a/changes/ticket34087 b/changes/ticket34087 new file mode 100644 index 0000000000..16990c305a --- /dev/null +++ b/changes/ticket34087 @@ -0,0 +1,3 @@ + o Minor bugfix (onion service v3 client): + - Remove a BUG() warning that can occur naturally. Fixes bug 34087; bugfix + on 0.3.2.1-alpha. diff --git a/src/feature/hs/hs_client.c b/src/feature/hs/hs_client.c index 80f4fb8484..09f0657fa8 100644 --- a/src/feature/hs/hs_client.c +++ b/src/feature/hs/hs_client.c @@ -1066,8 +1066,10 @@ close_or_reextend_intro_circ(origin_circuit_t *intro_circ) tor_assert(intro_circ); desc = hs_cache_lookup_as_client(&intro_circ->hs_ident->identity_pk); - if (BUG(desc == NULL)) { - /* We can't continue without a descriptor. */ + if (desc == NULL) { + /* We can't continue without a descriptor. This is possible if the cache + * was cleaned up between the intro point established and the reception of + * the introduce ack. */ goto close; } /* We still have the descriptor, great! Let's try to see if we can |