diff options
author | Nick Mathewson <nickm@torproject.org> | 2017-10-19 13:42:28 -0400 |
---|---|---|
committer | Nick Mathewson <nickm@torproject.org> | 2017-10-19 13:42:28 -0400 |
commit | 35746a9ee75608e7b2393e4519adca602bf2615f (patch) | |
tree | 3ed466046fe6cb7a0a6858c2b47765e2be459f3c /src/or/control.c | |
parent | 09562421589c055a6cb49c802a248f727390d158 (diff) | |
download | tor-35746a9ee75608e7b2393e4519adca602bf2615f.tar.gz tor-35746a9ee75608e7b2393e4519adca602bf2615f.zip |
Comment-only change: annotate exit() calls.
Sometimes when we call exit(), it's because the process is
completely hopeless: openssl has a broken AES-CTR implementation, or
the clock is in the 1960s, or something like that.
But sometimes, we should return cleanly from tor_main() instead, so
that embedders can keep embedding us and start another Tor process.
I've gone through all the exit() and _exit() calls to annotate them
with "exit ok" or "XXXX bad exit" -- the next step will be to fix
the bad exit()s.
First step towards 23848.
Diffstat (limited to 'src/or/control.c')
-rw-r--r-- | src/or/control.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/or/control.c b/src/or/control.c index 8173cb1e56..ab164700e6 100644 --- a/src/or/control.c +++ b/src/or/control.c @@ -6572,7 +6572,7 @@ monitor_owning_controller_process(const char *process_spec) msg); owning_controller_process_spec = NULL; tor_cleanup(); - exit(1); + exit(1); // XXXX bad exit: or questionable, at least. } } |