diff options
| author |   nicm <nicm@openbsd.org> | 2009-08-31 11:37:27 +0000 |
|---|---|---|
| committer | nicm <nicm@openbsd.org> | 2009-08-31 11:37:27 +0000 |
| commit | f369cd7f84b4cf77947ee420f9d50f37fa9795ab (patch) | |
| tree | f95a7a094b9b683669ba61d1ea1430b5e6b1e70c /usr.bin/tmux/server.c | |
| parent | Just use UINT_MAX as the maximum 4-byte AS number. It is good enough here. (diff) | |
| download | wireguard-openbsd-f369cd7f84b4cf77947ee420f9d50f37fa9795ab.tar.xz wireguard-openbsd-f369cd7f84b4cf77947ee420f9d50f37fa9795ab.zip | |
Don't call tty_free unless the client is a terminal, otherwise tty_init hasn't
been called and it may end up doing close(0). From Kalle Olavi Niemitalo.
Diffstat (limited to 'usr.bin/tmux/server.c')
| -rw-r--r-- | usr.bin/tmux/server.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/usr.bin/tmux/server.c b/usr.bin/tmux/server.c index c665a745389..33f5be07739 100644 --- a/usr.bin/tmux/server.c +++ b/usr.bin/tmux/server.c @@ -1,4 +1,4 @@ -/* $OpenBSD: server.c,v 1.24 2009/08/23 17:29:51 nicm Exp $ */ +/* $OpenBSD: server.c,v 1.25 2009/08/31 11:37:27 nicm Exp $ */ /* * Copyright (c) 2007 Nicholas Marriott <nicm@users.sourceforge.net> @@ -915,7 +915,12 @@ server_lost_client(struct client *c) } log_debug("lost client %d", c->ibuf.fd); - tty_free(&c->tty); + /* + * If CLIENT_TERMINAL hasn't been set, then tty_init hasn't been called + * and tty_free might close an unrelated fd. + */ + if (c->flags & CLIENT_TERMINAL) + tty_free(&c->tty); screen_free(&c->status); |