[tor-dev] When to exit 0 on error
nickm at alum.mit.edu
Sun Jun 25 00:03:22 UTC 2017
On Sat, Jun 24, 2017 at 12:38 AM, <f55jwk4f at vfemail.net> wrote:
> On Fri, Jun 23, 2017 at 10:29:32AM -0400, Zack Weinberg wrote:
>> On Fri, Jun 23, 2017 at 9:29 AM, Nick Mathewson <nickm at alum.mit.edu> wrote:
>> > On Thu, Jun 22, 2017 at 10:49 PM, <f55jwk4f at vfemail.net> wrote:
>> >> I noticed tor sometimes exits 0 on error
>> >> [...]
>> >> When to exit 0 on error?
>> > When the error is unrecoverable, and continuing would be worse then exiting.
>> I don't think the question was "when is it appropriate to _exit_ on
>> error?" I think it was "when is it appropriate to exit _0_ (i.e.
>> apparently successfully) on error?"
>> My expectation would be that exit(0) on fatal error is presumptively a
>> minor bug, it should be exit(1), and if there is some compelling
>> reason to exit successfully in a particular error case, there ought to
>> be a comment explaining that reason.
> Yes. I mean why to exit success instead of failure. But I don't know
> if those cases are typos, because there are 2 such cases, and a typo
> like that can be easily spotted in a review.
Ah, you're right. Those should be exit(1).
I've opened https://trac.torproject.org/projects/tor/ticket/22720 in
case anybody wants to fix this. This would be a good task for anybody
who's new to the Tor code: see doc/HACKING/GettingStarted.md for
information on getting started and writing patches.
More information about the tor-dev