hust testsuite: fix a false positive

[oweals/busybox.git] / miscutils / less.c

diff --git a/miscutils/less.c b/miscutils/less.c
index 877ab6ef2c2fadf8be6d6de1da4573cb408b8275..61acfdcb5045c06ae69b885f9c8d78f73c6dce6a 100644 (file)
--- a/miscutils/less.c
+++ b/miscutils/less.c
@@ -169,6 +169,7 @@ enum { pattern_valid = 0 };
 struct globals {
        int cur_fline; /* signed */
        int kbd_fd;  /* fd to get input from */
+       int kbd_fd_orig_flags;
        int less_gets_pos;
 /* last position in last line, taking into account tabs */
        size_t last_line_pos;
@@ -304,6 +305,8 @@ static void print_statusline(const char *str)
 static void less_exit(int code)
 {
        set_tty_cooked();
+       if (!(G.kbd_fd_orig_flags & O_NONBLOCK))
+               ndelay_off(kbd_fd);
        clear_line();
        if (code < 0)
                kill_myself_with_sig(- code); /* does not return */
@@ -1813,16 +1816,11 @@ int less_main(int argc, char **argv)
  try_ctty:
                tty_fd = open(CURRENT_TTY, O_RDONLY);
                if (tty_fd < 0) {
-                       /*
-                        * If all else fails, less 481 uses stdout. Mimic that.
-                        * Testcase where usually both ttyname(STDOUT_FILENO)
-                        * and open(CURRENT_TTY) fail:
-                        * su -s /bin/sh -c 'busybox less FILE' - nobody
-                        */
+                       /* If all else fails, less 481 uses stdout. Mimic that */
                        tty_fd = STDOUT_FILENO;
                }
        }
-       ndelay_on(tty_fd);
+       G.kbd_fd_orig_flags = ndelay_on(tty_fd);
        kbd_fd = tty_fd; /* save in a global */
 
        tcgetattr(kbd_fd, &term_orig);