Differences From Artifact [d04a91249e]:

• File unix/uxpty.c — part of check-in [37c4284d89] at 2013-02-23 15:00:29 on branch trunk — Don't forget to check the return values of setuid and friends. (user: simon size: 30048)

To Artifact [0dccf1a336]:

• File unix/uxpty.c — part of check-in [9613910863] at 2013-02-24 13:28:13 on branch trunk — Remove the half-hearted attempt to make the utmp helper process drop privileges just before dying of a fatal signal. I'm not sure what I intended it for in the first place; it certainly isn't doing its job properly (no setgid), it's causing compiler warnings due to not checking the setuid return code, and we can't think of any useful purpose for it. (user: simon size: 30026)

266
267
268
269
270
271
272
273
274
275
276
277
278
279
280

}

#ifndef OMIT_UTMP
static void fatal_sig_handler(int signum)
{
    putty_signal(signum, SIG_DFL);
    cleanup_utmp();
    setuid(getuid());
    raise(signum);
}
#endif

static int pty_open_slave(Pty pty)
{
    if (pty->slave_fd < 0) {

<

266
267
268
269
270
271
272

273
274
275
276
277
278
279

}

#ifndef OMIT_UTMP
static void fatal_sig_handler(int signum)
{
    putty_signal(signum, SIG_DFL);
    cleanup_utmp();

    raise(signum);
}
#endif

static int pty_open_slave(Pty pty)
{
    if (pty->slave_fd < 0) {