diff options
author | Alexander Færøy <ahf@torproject.org> | 2023-08-30 14:43:21 +0200 |
---|---|---|
committer | Alexander Færøy <ahf@torproject.org> | 2023-09-12 16:42:35 +0200 |
commit | 87fb8b641769688c88a6a23ed286ca44bfd989c1 (patch) | |
tree | 7f912edf6053dffdba31508e26cda231d8b1da3c /changes | |
parent | d6c89b1ae1b18cc3cae42638d7bbe1edd7e35715 (diff) | |
download | tor-87fb8b641769688c88a6a23ed286ca44bfd989c1.tar.gz tor-87fb8b641769688c88a6a23ed286ca44bfd989c1.zip |
Remove defensive assertion in `relay_key_is_unavailable_()`.
This patch removes a call to `tor_assert_nonfatal_unreached()` in
`relay_key_is_unavailable_()` that is only called when Tor is compiled
without relay support.
Unfortunately, the non-fatal assertion causes a BUG log
message to appear for clients when they start up without relay support
for each CPU worker we spawn. This makes it spotting issues during
bootstrap harder particularly for our iOS developers.
Since the call sites to `get_master_identity_key()` handles `NULL`
values already, we do not think this will be an issue later on.
Reported by Benjamin Erhart (@tla) from Guardian Project.
Fixes tpo/core/tor#40848.
Diffstat (limited to 'changes')
-rw-r--r-- | changes/ticket40848 | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/changes/ticket40848 b/changes/ticket40848 new file mode 100644 index 0000000000..a50a9a028f --- /dev/null +++ b/changes/ticket40848 @@ -0,0 +1,4 @@ + o Minor bugfix (defensive programming): + - Disable multiple BUG warnings of a missing relay identity key when + starting an instance of Tor compiled without relay support. + Fixes bug 40848; bugfix on 0.4.3.1-alpha. |