simon-svn: putty: simon

[Commits to Tartarus CVS repository.]

SVN root:       svn://svn.tartarus.org/sgt
Changes by:     simon
Revision:       9309
Date:           2011-09-19 17:38:23 +0100 (Mon, 19 Sep 2011)

Log message (8 lines):
We shouldn't fork off a utmp helper subprocess when we aren't setuid,
because (a) under that circumstance we won't be writing to utmp
anyway, and (b) if we aren't setuid, then we won't have created the
pty at the point we fork, so even if our subprocess _could_ have
written to utmp it wouldn't have done it right!

Spotted by valgrind (triggering on the access beyond the end of the
ttyname string in setup_utmp, clueing me in to it having been empty).

Modified files:
U   putty/unix/uxpty.c

Links:
http://svn.tartarus.org/sgt/?rev=9309&view=rev
http://svn.tartarus.org/sgt/putty/unix/uxpty.c?rev=9309&r1=9308&r2=9309