diff options
| author | George Kadianakis <desnacked@riseup.net> | 2017-03-01 12:38:12 +0200 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2017-03-01 08:46:53 -0500 |
| commit | 18a98206ede334c7332c43dc9f0b812611273f37 (patch) | |
| tree | af96c53e9caa51d6c41b9b51be6f5d8dca132afb /changes/bug21415 | |
| parent | 5ff31ff5b34fd69383b366d38e997822f0442086 (diff) | |
| download | tor-18a98206ede334c7332c43dc9f0b812611273f37.tar.gz tor-18a98206ede334c7332c43dc9f0b812611273f37.zip | |
Improve descriptor checks in the new guard algorithm.
- Make sure we check at least two guards for descriptor before making
circuits. We typically use the first primary guard for circuits, but
it can also happen that we use the second primary guard (e.g. if we
pick our first primary guard as an exit), so we should make sure we
have descriptors for both of them.
- Remove BUG() from the guard_has_descriptor() check since we now know
that this can happen in rare but legitimate situations as well, and we
should just move to the next guard in that case.
Diffstat (limited to 'changes/bug21415')
| -rw-r--r-- | changes/bug21415 | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/changes/bug21415 b/changes/bug21415 new file mode 100644 index 0000000000..f0aa72f81f --- /dev/null +++ b/changes/bug21415 @@ -0,0 +1,4 @@ + o Minor bugfix (entry guards): + - Silence a BUG() warning when attempting to use a guard whose descriptor + we don't know and make this scenario more unlikely to happen. Fixes bug + 21415; bugfix on 0.3.0.1-alpha. |