Opened 5 months ago

Closed 5 months ago

#21026 closed defect (fixed)

Remove the abort handler in the backtrace generator

Reported by: cypherpunks Owned by:
Priority: Medium Milestone: Tor: 0.3.0.x-final
Component: Core Tor/Tor Version: Tor:
Severity: Normal Keywords: review-group-14
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


The abort handler in the backtrace generator calls exit(3) which is not asynchronous-signal-safe. Calling exit(3) in this context is undefined behavior according to the C standard. See for more information.

FWICT the handler is meant to force the backtrace generator to always have an zero exit code. How the generator is currently used its exit code does not matter so the handler can probably be removed.

Patch coming soon after some testing.

Child Tickets

Attachments (1)

0001-Remove-abort-handler-from-the-backtrace-generator.patch (2.1 KB) - added by cypherpunks 5 months ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 5 months ago by cypherpunks

  • Status changed from new to needs_review
  • Version set to Tor:

The backtrace generator (and the abort handler) was added in commit 56e3f056e9a03015aa55392a8028e2cbe097a0fb which is Tor version

comment:2 Changed 5 months ago by nickm

  • Milestone changed from Tor: unspecified to Tor: 0.3.0.x-final

Honestly, I have no idea why I ever put an abort handler there.

comment:3 Changed 5 months ago by nickm

  • Keywords review-group-14 added

comment:4 Changed 5 months ago by nickm

  • Resolution set to fixed
  • Status changed from needs_review to closed


Note: See TracTickets for help on using tickets.