projects
/
u
/
mdw
/
putty
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Initialise 'psa' to NULL on every code path in the Pageant client
[u/mdw/putty]
/
unix
/
uxpty.c
diff --git
a/unix/uxpty.c
b/unix/uxpty.c
index
5ebd050
..
98f9179
100644
(file)
--- a/
unix/uxpty.c
+++ b/
unix/uxpty.c
@@
-373,15
+373,7
@@
static void pty_open_master(Pty pty)
strncpy(pty->name, ptsname(pty->master_fd), FILENAME_MAX-1);
#endif
strncpy(pty->name, ptsname(pty->master_fd), FILENAME_MAX-1);
#endif
- {
- /*
- * Set the pty master into non-blocking mode.
- */
- int fl;
- fl = fcntl(pty->master_fd, F_GETFL);
- if (fl != -1 && !(fl & O_NONBLOCK))
- fcntl(pty->master_fd, F_SETFL, fl | O_NONBLOCK);
- }
+ nonblock(pty->master_fd);
if (!ptys_by_fd)
ptys_by_fd = newtree234(pty_compare_by_fd);
if (!ptys_by_fd)
ptys_by_fd = newtree234(pty_compare_by_fd);
@@
-808,7
+800,7
@@
static const char *pty_init(void *frontend, void **backend_handle, Conf *conf,
}
close(pty->master_fd);
}
close(pty->master_fd);
- fcntl(slavefd, F_SETFD, 0); /* don't close on exec */
+ noncloexec(slavefd);
dup2(slavefd, 0);
dup2(slavefd, 1);
dup2(slavefd, 2);
dup2(slavefd, 0);
dup2(slavefd, 1);
dup2(slavefd, 2);
@@
-820,7
+812,11
@@
static const char *pty_init(void *frontend, void **backend_handle, Conf *conf,
pgrp = getpid();
tcsetpgrp(0, pgrp);
setpgid(pgrp, pgrp);
pgrp = getpid();
tcsetpgrp(0, pgrp);
setpgid(pgrp, pgrp);
- close(open(pty->name, O_WRONLY, 0));
+ {
+ int ptyfd = open(pty->name, O_WRONLY, 0);
+ if (ptyfd >= 0)
+ close(ptyfd);
+ }
setpgid(pgrp, pgrp);
{
char *term_env_var = dupprintf("TERM=%s",
setpgid(pgrp, pgrp);
{
char *term_env_var = dupprintf("TERM=%s",