[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-bugs] #15435 [Tor]: Tor should not close stdin on child processes.



#15435: Tor should not close stdin on child processes.
--------------------------------------+------------------------------------
 Reporter:  yawning                   |          Owner:
     Type:  enhancement               |         Status:  new
 Priority:  normal                    |      Milestone:  Tor: 0.2.7.x-final
Component:  Tor                       |        Version:  Tor: unspecified
 Keywords:  tor-pt, needs-spec-patch  |  Actual Points:
Parent ID:                            |         Points:
--------------------------------------+------------------------------------
 The lack of PTs having a reliable way to detect if tor is dead reared it's
 ugly head again in the form of #15434.  There aren't that many good ways
 to detect if a parent has died, beyond checking if stdin is still open
 (Proposed in #10047), but tor explicitly closes the child process's stdin.

 Since a variant of this has been working well, I propose that we make it
 official like thus:
  * Add a new enviornment variable of the form `PT_EXIT_ON_STDIN_CLOSE=1`,
 indicating that Tor will not close the stdin.
  * PTs SHOULD terminate as if they have received a SIGTERM, on stdin being
 closed if the environment variable is set.
  * All new tor versions will naturally set said env. var, and setup the pt
 plugin stdin correctly.

 This does mean that Bridges will consume one extra file descriptor per
 `ServerTransportPlugin` line in the torrc, but I view this as acceptable
 given the fact that this will bring some much needed sanity to our cleanup
 process.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/15435>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs