simon-svn: putty: simon

[tartarus-commits at lists.tartarus.org]

SVN root:       svn://ixion.tartarus.org/main
Changes by:     simon
Revision:       7107
Date:           2007-01-14 13:44:07 +0000 (Sun, 14 Jan 2007)

Log message (12 lines):
Remove the loops that close all open fds before running a
subprocess. They were intended to make sure the child process didn't
inherit anything embarrassing or inconvenient from us, such as the
master end of its own pty. This should fix Debian bug #357520.

(This doesn't seem to work _quite_ right in uxproxy.c's invocation
of a local proxy command: both ends of a GTK internal pipe end up in
the child process's fd space. This appears to be another GTK 1 bug,
inasmuch as it goes away when I build with Colin's preliminary GTK 2
patch; for the moment I think leaving that pipe lying around is
probably less harmful than hampering the proxy process's ability to
use extra fds by prior arrangement with PuTTY's parent process.)

Modified files:
U   putty/unix/uxproxy.c
U   putty/unix/uxpty.c

Links:
http://www.tartarus.org/~simon-anonsvn/viewcvs.cgi?rev=7107&view=rev
http://www.tartarus.org/~simon-anonsvn/viewcvs.cgi/putty/unix/uxproxy.c?rev=7107&r1=7106&r2=7107
http://www.tartarus.org/~simon-anonsvn/viewcvs.cgi/putty/unix/uxpty.c?rev=7107&r1=7106&r2=7107