Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#16766 closed enhancement (implemented)

Don't call process_signal directly from control.c

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

Description

It would be much smarter to use libevent events and event_active() to schedule them. (I found this problematic call by hand.)

Child Tickets

Change History (5)

comment:1 Changed 4 years ago by nickm

Initial code in my branch decouple_signals. Not working yet.

comment:2 Changed 4 years ago by nickm

Status: newneeds_review

Appears to be working now.

comment:3 Changed 4 years ago by yawning

This looks ok to me.

comment:4 Changed 4 years ago by nickm

Resolution: implemented
Status: needs_reviewclosed

Merged!

comment:5 Changed 4 years ago by isis

This also looks okay to me.

Note: See TracTickets for help on using tickets.