Opened 4 years ago

Closed 4 years ago

#16788 closed enhancement (implemented)

Don't call tor_cleanup() directly from lost_owning_controller()

Reported by: nickm Owned by:
Priority: Medium Milestone: Tor: 0.2.7.x-final
Component: Core Tor/Tor Version:
Severity: Keywords: SponsorS, TorCoreTeam201508, blob
Cc: Actual Points:
Parent ID: #16764 Points:
Reviewer: Sponsor:

Description

lost_owning_controller should just schedule our SIGTERM handler to get run. This will remove a tiny bit of duplicated code and simplify our callgraph

Child Tickets

Change History (3)

comment:1 Changed 4 years ago by nickm

Status: newneeds_review

see branch decouple_lost_owner branch, which is built on top of my decouple_signals branch from #16766 .

comment:2 Changed 4 years ago by yawning

lgtm.

comment:3 Changed 4 years ago by nickm

Resolution: implemented
Status: needs_reviewclosed

merged!

Note: See TracTickets for help on using tickets.