untrusted comment: verify with openbsd-66-base.pub
RWSvK/c+cFe24MrGJSj3IQJzRTrkTTt4o9CRESnhU7ROw08C6jmMGAtMRmcMB2UhWuuIlIZ34fCfoMTIOnThpRPtD+J259Ti+Qw=

OpenBSD 6.6 errata 034, July 16, 2020:

tty subsystem abuse can impact performance badly.

Apply by doing:
   signify -Vep /etc/signify/openbsd-66-base.pub -x 034_tty.patch.sig \
       -m - | (cd /usr/src && patch -p0)

And then rebuild and install a new kernel:
   KK=`sysctl -n kern.osversion | cut -d# -f1`
   cd /usr/src/sys/arch/`machine`/compile/$KK
   make obj
   make config
   make
   make install

Index: sys/sys/tty.h
===================================================================
RCS file: /cvs/src/sys/sys/tty.h,v
diff -u -p -u -r1.38 tty.h
--- sys/sys/tty.h       19 Jul 2019 00:17:16 -0000      1.38
+++ sys/sys/tty.h       14 Jul 2020 14:36:57 -0000
@@ -290,8 +290,8 @@ int  ttymodem(struct tty *tp, int flag);
int     ttyopen(dev_t device, struct tty *tp, struct proc *p);
int     ttyoutput(int c, struct tty *tp);
void    ttypend(struct tty *tp);
-void    ttyretype(struct tty *tp);
-void    ttyrub(int c, struct tty *tp);
+int     ttyretype(struct tty *tp);
+int     ttyrub(int c, struct tty *tp);
int     ttysleep(struct tty *tp, void *chan, int pri, char *wmesg);
int     ttywait(struct tty *tp);
int     ttywflush(struct tty *tp);
Index: sys/kern/tty.c
===================================================================
RCS file: /cvs/src/sys/kern/tty.c,v
retrieving revision 1.148
diff -u -p -u -r1.148 tty.c
--- sys/kern/tty.c      19 Jul 2019 00:17:15 -0000      1.148
+++ sys/kern/tty.c      14 Jul 2020 14:36:57 -0000
@@ -227,14 +227,15 @@ ttyclose(struct tty *tp)


/*
- * Process input of a single character received on a tty.
+ * Process input of a single character received on a tty.  Returns 0 for
+ * simple operations, 1 for costly ones (ptcwrite needs to know).
 */
int
ttyinput(int c, struct tty *tp)
{
       int iflag, lflag;
       u_char *cc;
-       int i, error;
+       int i, error, ret = 0;
       int s;

       enqueue_randomness(tp->t_dev << 8 | c);
@@ -342,7 +343,7 @@ parmrk:                             (void)putc(0377 | TTY_QUOTE,
                                       ttyflush(tp, FWRITE);
                                       ttyecho(c, tp);
                                       if (tp->t_rawq.c_cc + tp->t_canq.c_cc)
-                                               ttyretype(tp);
+                                               ret = ttyretype(tp);
                                       SET(tp->t_lflag, FLUSHO);
                               }
                               goto startoutput;
@@ -443,7 +444,7 @@ parmrk:                             (void)putc(0377 | TTY_QUOTE,
                */
               if (CCEQ(cc[VERASE], c)) {
                       if (tp->t_rawq.c_cc)
-                               ttyrub(unputc(&tp->t_rawq), tp);
+                               ret = ttyrub(unputc(&tp->t_rawq), tp);
                       goto endcase;
               }
               /*
@@ -452,10 +453,11 @@ parmrk:                           (void)putc(0377 | TTY_QUOTE,
               if (CCEQ(cc[VKILL], c)) {
                       if (ISSET(lflag, ECHOKE) &&
                           tp->t_rawq.c_cc == tp->t_rocount &&
-                           !ISSET(lflag, ECHOPRT))
+                           !ISSET(lflag, ECHOPRT)) {
                               while (tp->t_rawq.c_cc)
-                                       ttyrub(unputc(&tp->t_rawq), tp);
-                       else {
+                                       if (ttyrub(unputc(&tp->t_rawq), tp))
+                                               ret = 1;
+                       } else {
                               ttyecho(c, tp);
                               if (ISSET(lflag, ECHOK) ||
                                   ISSET(lflag, ECHOKE))
@@ -477,14 +479,16 @@ parmrk:                           (void)putc(0377 | TTY_QUOTE,
                        * erase whitespace
                        */
                       while ((c = unputc(&tp->t_rawq)) == ' ' || c == '\t')
-                               ttyrub(c, tp);
+                               if (ttyrub(c, tp))
+                                       ret = 1;
                       if (c == -1)
                               goto endcase;
                       /*
                        * erase last char of word and remember the
                        * next chars type (for ALTWERASE)
                        */
-                       ttyrub(c, tp);
+                       if (ttyrub(c, tp))
+                               ret = 1;
                       c = unputc(&tp->t_rawq);
                       if (c == -1)
                               goto endcase;
@@ -497,7 +501,8 @@ parmrk:                             (void)putc(0377 | TTY_QUOTE,
                        * erase rest of word
                        */
                       do {
-                               ttyrub(c, tp);
+                               if (ttyrub(c, tp))
+                                       ret = 1;
                               c = unputc(&tp->t_rawq);
                               if (c == -1)
                                       goto endcase;
@@ -510,7 +515,7 @@ parmrk:                             (void)putc(0377 | TTY_QUOTE,
                * reprint line (^R)
                */
               if (CCEQ(cc[VREPRINT], c) && ISSET(lflag, IEXTEN)) {
-                       ttyretype(tp);
+                       ret = ttyretype(tp);
                       goto endcase;
               }
               /*
@@ -577,12 +582,13 @@ endcase:
        */
       if (ISSET(tp->t_state, TS_TTSTOP) &&
           !ISSET(iflag, IXANY) && cc[VSTART] != cc[VSTOP])
-               return (0);
+               return (ret);
restartoutput:
       CLR(tp->t_lflag, FLUSHO);
       CLR(tp->t_state, TS_TTSTOP);
startoutput:
-       return (ttstart(tp));
+       ttstart(tp);
+       return (ret);
}

/*
@@ -1882,7 +1888,7 @@ ovhiwat:
 * Rubout one character from the rawq of tp
 * as cleanly as possible.
 */
-void
+int
ttyrub(int c, struct tty *tp)
{
       u_char *cp;
@@ -1890,15 +1896,14 @@ ttyrub(int c, struct tty *tp)
       int tabc, s;

       if (!ISSET(tp->t_lflag, ECHO) || ISSET(tp->t_lflag, EXTPROC))
-               return;
+               return 0;
       CLR(tp->t_lflag, FLUSHO);
       if (ISSET(tp->t_lflag, ECHOE)) {
               if (tp->t_rocount == 0) {
                       /*
                        * Screwed by ttwrite; retype
                        */
-                       ttyretype(tp);
-                       return;
+                       return ttyretype(tp);
               }
               if (c == ('\t' | TTY_QUOTE) || c == ('\n' | TTY_QUOTE))
                       ttyrubo(tp, 2);
@@ -1917,10 +1922,8 @@ ttyrub(int c, struct tty *tp)
                                       ttyrubo(tp, 2);
                               break;
                       case TAB:
-                               if (tp->t_rocount < tp->t_rawq.c_cc) {
-                                       ttyretype(tp);
-                                       return;
-                               }
+                               if (tp->t_rocount < tp->t_rawq.c_cc)
+                                       return ttyretype(tp);
                               s = spltty();
                               savecol = tp->t_column;
                               SET(tp->t_state, TS_CNTTB);
@@ -1958,6 +1961,7 @@ ttyrub(int c, struct tty *tp)
       } else
               ttyecho(tp->t_cc[VERASE], tp);
       --tp->t_rocount;
+       return 0;
}

/*
@@ -1979,7 +1983,7 @@ ttyrubo(struct tty *tp, int cnt)
 *     Reprint the rawq line.  Note, it is assumed that c_cc has already
 *     been checked.
 */
-void
+int
ttyretype(struct tty *tp)
{
       u_char *cp;
@@ -2001,6 +2005,11 @@ 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_nsec(tp, TTIPRI | PCATCH, "ttyretype", 1);
}

/*
Index: sys/kern/tty_pty.c
===================================================================
RCS file: /cvs/src/sys/kern/tty_pty.c,v
diff -u -p -u -r1.94 tty_pty.c
--- sys/kern/tty_pty.c  19 Jul 2019 00:17:16 -0000      1.94
+++ sys/kern/tty_pty.c  14 Jul 2020 14:36:57 -0000
@@ -564,7 +564,8 @@ again:
                               wakeup(&tp->t_rawq);
                               goto block;
                       }
-                       (*linesw[tp->t_line].l_rint)(*cp++, tp);
+                       if ((*linesw[tp->t_line].l_rint)(*cp++, tp) == EINTR)
+                               goto interrupt;
                       cnt++;
                       cc--;
               }
@@ -592,6 +593,7 @@ block:
       if (error == 0)
               goto again;

+interrupt:
       /* adjust for data copied in but not written */
       uio->uio_resid += cc;
done:

You are viewing proxied material from ftp.icm.edu.pl. The copyright of proxied material belongs to its original authors. Any comments or complaints in relation to proxied material should be directed to the original authors of the content concerned. Please see the disclaimer for more details.