diff options
| author |   deraadt <deraadt@openbsd.org> | 2020-07-20 14:34:16 +0000 |
|---|---|---|
| committer | deraadt <deraadt@openbsd.org> | 2020-07-20 14:34:16 +0000 |
| commit | 2df268a15789180a6dc2e6ac48bb86caa4610545 (patch) | |
| tree | 9c8ff0ad32ec9a3cfaa8205d5ef76931abaa583f /sys/kern/tty.c | |
| parent | Switch the default pager from "more -s" to "less". (diff) | |
| download | wireguard-openbsd-2df268a15789180a6dc2e6ac48bb86caa4610545.tar.xz wireguard-openbsd-2df268a15789180a6dc2e6ac48bb86caa4610545.zip | |
Sigh. Only the ptyc case should tsleep in ttyretype, since others can
arrive in the wrong context. Found by jcs.
Diffstat (limited to 'sys/kern/tty.c')
| -rw-r--r-- | sys/kern/tty.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/sys/kern/tty.c b/sys/kern/tty.c index e71ca0e4fb4..33f6c4baf71 100644 --- a/sys/kern/tty.c +++ b/sys/kern/tty.c @@ -1,4 +1,4 @@ -/* $OpenBSD: tty.c,v 1.160 2020/07/15 02:29:26 deraadt Exp $ */ +/* $OpenBSD: tty.c,v 1.161 2020/07/20 14:34:16 deraadt Exp $ */ /* $NetBSD: tty.c,v 1.68.4.2 1996/06/06 16:04:52 thorpej Exp $ */ /*- @@ -228,8 +228,8 @@ ttyclose(struct tty *tp) /* - * Process input of a single character received on a tty. Returns 0 for - * simple operations, 1 for costly ones (ptcwrite needs to know). + * Process input of a single character received on a tty. Returns 0 normally, + * 1 if a costly operation was reached. */ int ttyinput(int c, struct tty *tp) @@ -2019,11 +2019,7 @@ ttyretype(struct tty *tp) tp->t_rocount = tp->t_rawq.c_cc; tp->t_rocol = 0; - /* - * Yield because of expense, or possible ptcwrite() injection flood. - * Also check for interrupt, and return upwards. - */ - return tsleep(tp, TTIPRI | PCATCH, "ttyretype", 1); + return (1); } /* |