[tor-bugs] #3049 [Tor Client]: Allow a Tor process to be ‘owned’ by a controller process
Tor Bug Tracker & Wiki
torproject-admin at torproject.org
Wed May 4 03:48:32 UTC 2011
#3049: Allow a Tor process to be ‘owned’ by a controller process
-------------------------+--------------------------------------------------
Reporter: rransom | Owner:
Type: enhancement | Status: new
Priority: major | Milestone: Tor: 0.2.2.x-final
Component: Tor Client | Version:
Keywords: | Parent:
Points: | Actualpoints:
-------------------------+--------------------------------------------------
Comment(by rransom):
Replying to [comment:1 chiiph]:
> So, TAKEOWNERSHIP will be used when the process doesn't have
OwningControllerPID option set? There may be some conflicts there, or will
the TAKEOWNERSHIP set OwningControllerPID and fail if the latter is
already set?
The TAKEOWNERSHIP command and OwningControllerPID options should do
exactly what I specified that they should do in the ticket ‘description’.
There is no conflict between their effects.
When Vidalia is configured to ‘own’ the Tor instance it controls, Vidalia
should specify its process ID to Tor using the OwningControllerPID option,
and then send the TAKEOWNERSHIP command when it connects to Tor's control
port. After Vidalia has sent the TAKEOWNERSHIP command, it should unset
the OwningControllerPID option so that Tor will stop polling for a process
with that process ID.
The reason to have an OwningControllerPID option is to try to avoid
leaving a zombie Tor process if Vidalia exits before connecting to Tor's
control port. The reason to have a TAKEOWNERSHIP command is to reliably
avoid leaving a zombie Tor process if Vidalia exits after connecting to
Tor's control port, even if the OS gives a new process the same process ID
that Vidalia had before Tor polls for it again.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/3049#comment:2>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the tor-bugs
mailing list