diff options
| author | Sebastian Hahn <sebastian@torproject.org> | 2011-05-21 19:10:11 +0200 |
|---|---|---|
| committer | Roger Dingledine <arma@torproject.org> | 2011-05-21 16:08:21 -0400 |
| commit | 3ff7925a7027b81cb9f14b0863972b4d4e5b46a4 (patch) | |
| tree | 29d0fcd6fb70d09fd26fe23b255e677567cc8ac4 /src/or/config.c | |
| parent | b8ffb00cf1ddf4ff1b64335f8677d600a948fffc (diff) | |
| download | tor-3ff7925a7027b81cb9f14b0863972b4d4e5b46a4.tar tor-3ff7925a7027b81cb9f14b0863972b4d4e5b46a4.tar.gz | |
Don't recreate descriptor on sighup
We used to regenerate our descriptor whenever we'd get a sighup. This
was caused by a bug in options_transition_affects_workers() that would
return true even if the options were exactly the same. Down the call
path we'd call init_keys(), which made us make a new descriptor which
the authorities would reject, and the node would subsequently fall out
of the consensus.
This patch fixes only the first part of this bug:
options_transition_affects_workers() behaves correctly now. The second
part still wants a fix.
Diffstat (limited to 'src/or/config.c')
| -rw-r--r-- | src/or/config.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/or/config.c b/src/or/config.c index 68a6b29f1..36fb991e3 100644 --- a/src/or/config.c +++ b/src/or/config.c @@ -3809,7 +3809,7 @@ options_transition_affects_workers(or_options_t *old_options, old_options->ORPort != new_options->ORPort || old_options->ServerDNSSearchDomains != new_options->ServerDNSSearchDomains || - old_options->SafeLogging != new_options->SafeLogging || + old_options->_SafeLogging != new_options->_SafeLogging || old_options->ClientOnly != new_options->ClientOnly || public_server_mode(old_options) != public_server_mode(new_options) || !config_lines_eq(old_options->Logs, new_options->Logs) || |