]> git.ipfire.org Git - thirdparty/bash.git/blobdiff - builtins/read.def
Bash-5.2 patch 11: reset readline timeout after read -e -t
[thirdparty/bash.git] / builtins / read.def
index 8fc5647638a4536f3da36253c2d32fd67f98c708..ddd91d32d5db90d341f3b9173d07719232826704 100644 (file)
@@ -1,7 +1,7 @@
 This file is read.def, from which is created read.c.
 It implements the builtin "read" in Bash.
 
-Copyright (C) 1987-2009 Free Software Foundation, Inc.
+Copyright (C) 1987-2021 Free Software Foundation, Inc.
 
 This file is part of GNU Bash, the Bourne Again SHell.
 
@@ -22,7 +22,7 @@ $PRODUCES read.c
 
 $BUILTIN read
 $FUNCTION read_builtin
-$SHORT_DOC read [-ers] [-a array] [-d delim] [-i text] [-n nchars] [-p prompt] [-t timeout] [-u fd] [name ...]
+$SHORT_DOC read [-ers] [-a array] [-d delim] [-i text] [-n nchars] [-N nchars] [-p prompt] [-t timeout] [-u fd] [name ...]
 Read a line from the standard input and split it into fields.
 
 Reads a single line from the standard input, or from file descriptor FD
@@ -30,7 +30,8 @@ if the -u option is supplied.  The line is split into fields as with word
 splitting, and the first word is assigned to the first NAME, the second
 word to the second NAME, and so on, with any leftover words assigned to
 the last NAME.  Only the characters found in $IFS are recognized as word
-delimiters.
+delimiters. By default, the backslash character escapes delimiter characters
+and newline.
 
 If no NAMEs are supplied, the line read is stored in the REPLY variable.
 
@@ -39,24 +40,31 @@ Options:
                variable ARRAY, starting at zero
   -d delim     continue until the first character of DELIM is read, rather
                than newline
-  -e           use Readline to obtain the line in an interactive shell
-  -i text      Use TEXT as the initial text for Readline
+  -e   use Readline to obtain the line
+  -i text      use TEXT as the initial text for Readline
   -n nchars    return after reading NCHARS characters rather than waiting
-               for a newline
+               for a newline, but honor a delimiter if fewer than
+               NCHARS characters are read before the delimiter
+  -N nchars    return only after reading exactly NCHARS characters, unless
+               EOF is encountered or read times out, ignoring any
+               delimiter
   -p prompt    output the string PROMPT without a trailing newline before
                attempting to read
-  -r           do not allow backslashes to escape any characters
-  -s           do not echo input coming from a terminal
-  -t timeout   time out and return failure if a complete line of input is
-               not read withint TIMEOUT seconds.  The value of the TMOUT
-               variable is the default timeout.  TIMEOUT may be a
-               fractional number.  If TIMEOUT is 0, read returns success only
-               if input is available on the specified file descriptor.  The
-               exit status is greater than 128 if the timeout is exceeded
-  -u fd                read from file descriptor FD instead of the standard input
+  -r   do not allow backslashes to escape any characters
+  -s   do not echo input coming from a terminal
+  -t timeout   time out and return failure if a complete line of
+               input is not read within TIMEOUT seconds.  The value of the
+               TMOUT variable is the default timeout.  TIMEOUT may be a
+               fractional number.  If TIMEOUT is 0, read returns
+               immediately, without trying to read any data, returning
+               success only if input is available on the specified
+               file descriptor.  The exit status is greater than 128
+               if the timeout is exceeded
+  -u fd        read from file descriptor FD instead of the standard input
 
 Exit Status:
-The return code is zero, unless end-of-file is encountered, read times out,
+The return code is zero, unless end-of-file is encountered, read times out
+(in which case it's greater than 128), a variable assignment error occurs,
 or an invalid file descriptor is supplied as the argument to -u.
 $END
 
@@ -86,6 +94,7 @@ $END
 #include "../shell.h"
 #include "common.h"
 #include "bashgetopt.h"
+#include "trap.h"
 
 #include <shtty.h>
 
@@ -98,6 +107,9 @@ $END
 #  include "input.h"
 #endif
 
+#include "shmbutil.h"
+#include "timer.h"
+
 #if !defined(errno)
 extern int errno;
 #endif
@@ -105,41 +117,80 @@ extern int errno;
 struct ttsave
 {
   int fd;
-  TTYSTRUCT *attrs;
+  TTYSTRUCT attrs;
 };
 
 #if defined (READLINE)
-static void reset_attempted_completion_function __P((char *));
-static int set_itext __P((void));
-static char *edit_line __P((char *, char *));
-static void set_eol_delim __P((int));
-static void reset_eol_delim __P((char *));
+static void reset_attempted_completion_function PARAMS((char *));
+static int set_itext PARAMS((void));
+static char *edit_line PARAMS((char *, char *));
+static void set_eol_delim PARAMS((int));
+static void reset_eol_delim PARAMS((char *));
+static void set_readline_timeout PARAMS((sh_timer *t, time_t, long));
 #endif
-static SHELL_VAR *bind_read_variable __P((char *, char *));
+static SHELL_VAR *bind_read_variable PARAMS((char *, char *, int));
 #if defined (HANDLE_MULTIBYTE)
-static int read_mbchar __P((int, char *, int, int, int));
+static int read_mbchar PARAMS((int, char *, int, int, int));
 #endif
-static void ttyrestore __P((struct ttsave *));
+static void ttyrestore PARAMS((struct ttsave *));
+
+static sighandler sigalrm PARAMS((int));
+static void reset_timeout PARAMS((void));
 
-static sighandler sigalrm __P((int));
-static void reset_alarm __P((void));
+/* Try this to see what the rest of the shell can do with the information. */
+sh_timer *read_timeout;
 
-static procenv_t alrmbuf;
+static int reading, tty_modified;
 static SigHandler *old_alrm;
 static unsigned char delim;
 
+static struct ttsave termsave;
+
+/* In all cases, SIGALRM just sets a flag that we check periodically.  This
+   avoids problems with the semi-tricky stuff we do with the xfree of
+   input_string at the top of the unwind-protect list (see below). */
+
+/* Set a flag that check_read_timeout can check.  This relies on zread or
+   read_builtin calling trap.c:check_signals() (which calls check_read_timeout()) */
 static sighandler
 sigalrm (s)
      int s;
 {
-  longjmp (alrmbuf, 1);
+  /* Display warning if this is called without read_timeout set? */
+  if (read_timeout)
+    read_timeout->alrmflag = 1;
 }
 
 static void
-reset_alarm ()
+reset_timeout ()
+{
+  /* Cancel alarm before restoring signal handler. */
+  if (read_timeout)
+    shtimer_clear (read_timeout);
+#if defined (READLINE)
+  rl_clear_timeout ();
+#endif
+  read_timeout = 0;
+}
+
+void
+check_read_timeout ()
 {
-  set_signal_handler (SIGALRM, old_alrm);
-  falarm (0, 0);
+  if (read_timeout && shtimer_chktimeout (read_timeout))
+    sh_longjmp (read_timeout->jmpenv, 1);
+}
+
+int
+read_builtin_timeout (fd)
+     int fd;
+{
+  if ((read_timeout == 0) ||
+      (read_timeout->fd != fd) ||
+      (read_timeout->tmout.tv_sec == 0 && read_timeout->tmout.tv_usec == 0))
+    return 0;
+
+  return ((read_timeout->flags & SHTIMER_ALARM) ? shtimer_alrm (read_timeout)
+                                               : shtimer_select (read_timeout));
 }
 
 /* Read the value of the shell variables whose names follow.
@@ -153,9 +204,12 @@ read_builtin (list)
      WORD_LIST *list;
 {
   register char *varname;
-  int size, i, nr, pass_next, saw_escape, eof, opt, retval, code, print_ps2;
+  int size, nr, pass_next, saw_escape, eof, opt, retval, code, print_ps2, nflag;
+  volatile int i;
   int input_is_tty, input_is_pipe, unbuffered_read, skip_ctlesc, skip_ctlnul;
-  int raw, edit, nchars, silent, have_timeout, fd;
+  int raw, edit, nchars, silent, have_timeout, ignore_delim, fd;
+  int lastsig, t_errno;
+  int mb_cur_max;
   unsigned int tmsec, tmusec;
   long ival, uval;
   intmax_t intval;
@@ -165,13 +219,16 @@ read_builtin (list)
   struct stat tsb;
   SHELL_VAR *var;
   TTYSTRUCT ttattrs, ttset;
-  struct ttsave termsave;
+  sigset_t chldset, prevset;
 #if defined (ARRAY_VARS)
   WORD_LIST *alist;
+  int vflags;
 #endif
+  int bindflags;
 #if defined (READLINE)
   char *rlbuf, *itext;
   int rlind;
+  FILE *save_instream;
 #endif
 
   USE_VAR(size);
@@ -196,6 +253,10 @@ read_builtin (list)
 #endif
   USE_VAR(list);
   USE_VAR(ps2);
+  USE_VAR(lastsig);
+
+  reading = tty_modified = 0;
+  read_timeout = 0;
 
   i = 0;               /* Index into the string that we are reading. */
   raw = edit = 0;      /* Not reading raw input by default. */
@@ -208,12 +269,14 @@ read_builtin (list)
   rlind = 0;
 #endif
 
+  mb_cur_max = MB_CUR_MAX;
   tmsec = tmusec = 0;          /* no timeout */
   nr = nchars = input_is_tty = input_is_pipe = unbuffered_read = have_timeout = 0;
   delim = '\n';                /* read until newline */
+  ignore_delim = nflag = 0;
 
   reset_internal_getopt ();
-  while ((opt = internal_getopt (list, "ersa:d:i:n:p:t:u:")) != -1)
+  while ((opt = internal_getopt (list, "ersa:d:i:n:p:t:u:N:")) != -1)
     {
       switch (opt)
        {
@@ -242,7 +305,7 @@ read_builtin (list)
          break;
 #endif
        case 't':
-         code = uconvert (list_optarg, &ival, &uval);
+         code = uconvert (list_optarg, &ival, &uval, (char **)NULL);
          if (code == 0 || ival < 0 || uval < 0)
            {
              builtin_error (_("%s: invalid timeout specification"), list_optarg);
@@ -255,7 +318,11 @@ read_builtin (list)
              tmusec = uval;
            }
          break;
+       case 'N':
+         ignore_delim = 1;
+         delim = -1;
        case 'n':
+         nflag = 1;
          code = legal_number (list_optarg, &intval);
          if (code == 0 || intval < 0 || intval != (int)intval)
            {
@@ -283,6 +350,7 @@ read_builtin (list)
        case 'd':
          delim = *list_optarg;
          break;
+       CASE_HELPOPT;
        default:
          builtin_usage ();
          return (EX_USAGE);
@@ -293,26 +361,52 @@ read_builtin (list)
   /* `read -t 0 var' tests whether input is available with select/FIONREAD,
      and fails if those are unavailable */
   if (have_timeout && tmsec == 0 && tmusec == 0)
-#if 0
-    return (EXECUTION_FAILURE);
-#else
     return (input_avail (fd) ? EXECUTION_SUCCESS : EXECUTION_FAILURE);
+
+  /* Convenience: check early whether or not the first of possibly several
+     variable names is a valid identifier, and bail early if so. */
+#if defined (ARRAY_VARS)
+  if (list)
+    SET_VFLAGS (list->word->flags, vflags, bindflags);
+  if (list && legal_identifier (list->word->word) == 0 && valid_array_reference (list->word->word, vflags) == 0)
+#else
+  bindflags = 0;
+  if (list && legal_identifier (list->word->word) == 0)
 #endif
+    {
+      sh_invalidid (list->word->word);
+      return (EXECUTION_FAILURE);
+    }
+
+  /* If we're asked to ignore the delimiter, make sure we do. */
+  if (ignore_delim)
+    delim = -1;
 
   /* IF IFS is unset, we use the default of " \t\n". */
   ifs_chars = getifs ();
   if (ifs_chars == 0)          /* XXX - shouldn't happen */
     ifs_chars = "";
+  /* If we want to read exactly NCHARS chars, don't split on IFS */
+  if (ignore_delim)
+    ifs_chars = "";
   for (skip_ctlesc = skip_ctlnul = 0, e = ifs_chars; *e; e++)
     skip_ctlesc |= *e == CTLESC, skip_ctlnul |= *e == CTLNUL;
 
   input_string = (char *)xmalloc (size = 112); /* XXX was 128 */
   input_string[0] = '\0';
 
+  /* More input and options validation */
+  if (nflag == 1 && nchars == 0)
+    {
+      retval = read (fd, &c, 0);
+      retval = (retval >= 0) ? EXECUTION_SUCCESS : EXECUTION_FAILURE;
+      goto assign_vars;                /* bail early if asked to read 0 chars */
+    }
+
   /* $TMOUT, if set, is the default timeout for read. */
   if (have_timeout == 0 && (e = get_string_value ("TMOUT")))
     {
-      code = uconvert (e, &ival, &uval);
+      code = uconvert (e, &ival, &uval, (char **)NULL);
       if (code == 0 || ival < 0 || uval < 0)
        tmsec = tmusec = 0;
       else
@@ -322,6 +416,12 @@ read_builtin (list)
        }
     }
 
+#if defined (SIGCHLD)
+  sigemptyset (&chldset);
+  sigprocmask (SIG_BLOCK, (sigset_t *)0, &chldset);
+  sigaddset (&chldset, SIGCHLD);
+#endif
+
   begin_unwind_frame ("read_builtin");
 
 #if defined (BUFFERED_INPUT)
@@ -329,7 +429,11 @@ read_builtin (list)
     sync_buffered_stream (default_buffered_input);
 #endif
 
+#if 1
   input_is_tty = isatty (fd);
+#else
+  input_is_tty = 1;
+#endif
   if (input_is_tty == 0)
 #ifndef __CYGWIN__
     input_is_pipe = (lseek (fd, 0L, SEEK_CUR) < 0) && (errno == ESPIPE);
@@ -366,25 +470,59 @@ read_builtin (list)
 
   if (tmsec > 0 || tmusec > 0)
     {
-      code = setjmp (alrmbuf);
+      read_timeout = shtimer_alloc ();
+      read_timeout->flags = SHTIMER_LONGJMP;
+
+#if defined (HAVE_SELECT)
+      read_timeout->flags |= (edit || posixly_correct) ? SHTIMER_ALARM : SHTIMER_SELECT;
+#else
+      read_timeout->flags |= SHTIMER_ALARM;
+#endif
+      read_timeout->fd = fd;
+
+      read_timeout->alrm_handler = sigalrm;
+    }
+
+  if (tmsec > 0 || tmusec > 0)
+    {
+      code = setjmp_nosigs (read_timeout->jmpenv);
       if (code)
        {
+         reset_timeout ();
+         sigprocmask (SIG_SETMASK, &prevset, (sigset_t *)0);
+
          /* Tricky.  The top of the unwind-protect stack is the free of
             input_string.  We want to run all the rest and use input_string,
-            so we have to remove it from the stack. */
-         remove_unwind_protect ();
-         run_unwind_frame ("read_builtin");
+            so we have to save input_string temporarily, run the unwind-
+            protects, then restore input_string so we can use it later */
+         orig_input_string = 0;
          input_string[i] = '\0';       /* make sure it's terminated */
+         if (i == 0)
+           {
+             t = (char *)xmalloc (1);
+             t[0] = 0;
+           }
+         else
+           t = savestring (input_string);
+
+         run_unwind_frame ("read_builtin");
+         input_string = t;
          retval = 128+SIGALRM;
          goto assign_vars;
        }
-      old_alrm = set_signal_handler (SIGALRM, sigalrm);
-      add_unwind_protect (reset_alarm, (char *)NULL);
+      if (interactive_shell == 0)
+       initialize_terminating_signals ();
+      add_unwind_protect (reset_timeout, (char *)NULL);
 #if defined (READLINE)
       if (edit)
-       add_unwind_protect (reset_attempted_completion_function, (char *)NULL);
+       {
+         add_unwind_protect (reset_attempted_completion_function, (char *)NULL);
+         add_unwind_protect (bashline_reset_event_hook, (char *)NULL);
+         set_readline_timeout (read_timeout, tmsec, tmusec);
+       }
+      else
 #endif
-      falarm (tmsec, tmusec);
+      shtimer_set (read_timeout, tmsec, tmusec);
     }
 
   /* If we've been asked to read only NCHARS chars, or we're using some
@@ -413,13 +551,16 @@ read_builtin (list)
          /* ttsave() */
          termsave.fd = fd;
          ttgetattr (fd, &ttattrs);
-         termsave.attrs = &ttattrs;
+         termsave.attrs = ttattrs;
 
          ttset = ttattrs;        
          i = silent ? ttfd_cbreak (fd, &ttset) : ttfd_onechar (fd, &ttset);
          if (i < 0)
            sh_ttyerror (1);
+         tty_modified = 1;
          add_unwind_protect ((Function *)ttyrestore, (char *)&termsave);
+         if (interactive_shell == 0)
+           initialize_terminating_signals ();
        }
     }
   else if (silent)     /* turn off echo but leave term in canonical mode */
@@ -427,24 +568,46 @@ read_builtin (list)
       /* ttsave (); */
       termsave.fd = fd;
       ttgetattr (fd, &ttattrs);
-      termsave.attrs = &ttattrs;
+      termsave.attrs = ttattrs;
 
       ttset = ttattrs;
       i = ttfd_noecho (fd, &ttset);                    /* ttnoecho (); */
       if (i < 0)
        sh_ttyerror (1);
 
+      tty_modified = 1;
       add_unwind_protect ((Function *)ttyrestore, (char *)&termsave);
+      if (interactive_shell == 0)
+       initialize_terminating_signals ();
     }
 
+#if defined (READLINE)
+  save_instream = 0;
+  if (edit && fd != 0)
+    {
+      if (bash_readline_initialized == 0)
+       initialize_readline ();
+
+      unwind_protect_var (rl_instream);
+      save_instream = rl_instream;
+      rl_instream = fdopen (fd, "r");  
+    }
+#endif
+
   /* This *must* be the top unwind-protect on the stack, so the manipulation
      of the unwind-protect stack after the realloc() works right. */
   add_unwind_protect (xfree, input_string);
-  interrupt_immediately++;
-  terminate_immediately++;
-
-  unbuffered_read = (nchars > 0) || (delim != '\n') || input_is_pipe;
 
+  check_read_timeout ();
+  /* These only matter if edit == 0 */
+  if ((nchars > 0) && (input_is_tty == 0) && ignore_delim)     /* read -N */
+    unbuffered_read = 2;
+#if 0
+  else if ((nchars > 0) || (delim != '\n') || input_is_pipe)
+#else
+  else if (((nchars > 0 || delim != '\n') && input_is_tty) || input_is_pipe)
+    unbuffered_read = 1;
+#endif
   if (prompt && edit == 0)
     {
       fprintf (stderr, "%s", prompt);
@@ -458,19 +621,32 @@ read_builtin (list)
   ps2 = 0;
   for (print_ps2 = eof = retval = 0;;)
     {
+      check_read_timeout ();
+
 #if defined (READLINE)
       if (edit)
        {
-         if (rlbuf && rlbuf[rlind] == '\0')
+         /* If we have a null delimiter, don't treat NULL as ending the line */
+         if (rlbuf && rlbuf[rlind] == '\0' && delim != '\0')
            {
-             xfree (rlbuf);
+             free (rlbuf);
              rlbuf = (char *)0;
            }
+#if defined (SIGCHLD)
+         if (tmsec > 0 || tmusec > 0)
+           sigprocmask (SIG_SETMASK, &chldset, &prevset);
+#endif
          if (rlbuf == 0)
            {
+             reading = 1;
              rlbuf = edit_line (prompt ? prompt : "", itext);
+             reading = 0;
              rlind = 0;
            }
+#if defined (SIGCHLD)
+         if (tmsec > 0 || tmusec > 0)
+           sigprocmask (SIG_SETMASK, &prevset, (sigset_t *)0);
+#endif
          if (rlbuf == 0)
            {
              eof = 1;
@@ -491,26 +667,73 @@ read_builtin (list)
          print_ps2 = 0;
        }
 
-      if (unbuffered_read)
-       retval = zread (fd, &c, 1);
+      reading = 1;
+      check_read_timeout ();
+      errno = 0;
+
+#if defined (SIGCHLD)
+      if (tmsec > 0 || tmusec > 0)
+       sigprocmask (SIG_SETMASK, &chldset, &prevset);
+#endif
+      if (unbuffered_read == 2)
+       retval = posixly_correct ? zreadintr (fd, &c, 1) : zreadn (fd, &c, nchars - nr);
+      else if (unbuffered_read)
+       retval = posixly_correct ? zreadintr (fd, &c, 1) : zread (fd, &c, 1);
       else
-       retval = zreadc (fd, &c);
+       retval = posixly_correct ? zreadcintr (fd, &c) : zreadc (fd, &c);
+#if defined (SIGCHLD)
+      if (tmsec > 0 || tmusec > 0)
+       sigprocmask (SIG_SETMASK, &prevset, (sigset_t *)0);
+#endif
+
+      reading = 0;
 
       if (retval <= 0)
        {
+         int t;
+
+         t = errno;
+         if (retval < 0 && errno == EINTR)
+           {
+             check_signals ();         /* in case we didn't call zread via zreadc */
+             lastsig = LASTSIG();
+             if (lastsig == 0)
+               lastsig = trapped_signal_received;
+#if 0
+             run_pending_traps ();     /* because interrupt_immediately is not set */
+#endif
+           }
+         else
+           lastsig = 0;
+         if (terminating_signal && tty_modified)
+           ttyrestore (&termsave);     /* fix terminal before exiting */
+         CHECK_TERMSIG;
          eof = 1;
+         errno = t;    /* preserve it for the error message below */
          break;
        }
 
+      QUIT;            /* in case we didn't call check_signals() */
 #if defined (READLINE)
        }
 #endif
 
-      if (i + 4 >= size)       /* XXX was i + 2; use i + 4 for multibyte/read_mbchar */
+      if (retval <= 0)                 /* XXX shouldn't happen */
+       check_read_timeout ();
+
+      /* XXX -- use i + mb_cur_max (at least 4) for multibyte/read_mbchar */
+      if (i + (mb_cur_max > 4 ? mb_cur_max : 4) >= size)
        {
-         input_string = (char *)xrealloc (input_string, size += 128);
-         remove_unwind_protect ();
-         add_unwind_protect (xfree, input_string);
+         char *t;
+         t = (char *)xrealloc (input_string, size += 128);
+
+         /* Only need to change unwind-protect if input_string changes */
+         if (t != input_string)
+           {
+             input_string = t;
+             remove_unwind_protect ();
+             add_unwind_protect (xfree, input_string);
+           }
        }
 
       /* If the next character is to be accepted verbatim, a backslash
@@ -520,7 +743,8 @@ read_builtin (list)
          pass_next = 0;
          if (c == '\n')
            {
-             i--;              /* back up over the CTLESC */
+             if (skip_ctlesc == 0 && i > 0)
+               i--;            /* back up over the CTLESC */
              if (interactive && input_is_tty && raw == 0)
                print_ps2 = 1;
            }
@@ -541,9 +765,12 @@ read_builtin (list)
          continue;
        }
 
-      if ((unsigned char)c == delim)
+      if (ignore_delim == 0 && (unsigned char)c == delim)
        break;
 
+      if (c == '\0' && delim != '\0')
+       continue;               /* skip NUL bytes in input */
+
       if ((skip_ctlesc == 0 && c == CTLESC) || (skip_ctlnul == 0 && c == CTLNUL))
        {
          saw_escape++;
@@ -552,12 +779,34 @@ read_builtin (list)
 
 add_char:
       input_string[i++] = c;
+      check_read_timeout ();
 
 #if defined (HANDLE_MULTIBYTE)
-      if (nchars > 0 && MB_CUR_MAX > 1)
+      /* XXX - what if C == 127? Can DEL introduce a multibyte sequence? */
+      if (mb_cur_max > 1 && is_basic (c) == 0)
        {
          input_string[i] = '\0';       /* for simplicity and debugging */
-         i += read_mbchar (fd, input_string, i, c, unbuffered_read);
+         /* If we got input from readline, grab the next multibyte char from
+            rlbuf. */
+#  if defined (READLINE)
+         if (edit)
+           {
+             size_t clen;
+             clen = mbrlen (rlbuf + rlind - 1, mb_cur_max, (mbstate_t *)NULL);
+             /* We only deal with valid multibyte sequences longer than one
+                byte. If we get anything else, we leave the one character
+                copied and move on to the next. */
+             if ((int)clen > 1)
+               {
+                 memcpy (input_string+i, rlbuf+rlind, clen-1);
+                 i += clen - 1;
+                 rlind += clen - 1;
+               }
+           }
+         else
+#  endif
+         if (locale_utf8locale == 0 || ((c & 0x80) != 0))
+           i += read_mbchar (fd, input_string, i, c, unbuffered_read);
        }
 #endif
 
@@ -567,18 +816,24 @@ add_char:
        break;
     }
   input_string[i] = '\0';
+  check_read_timeout ();
+
+#if defined (READLINE)
+  if (edit)
+    free (rlbuf);
+#endif
 
-#if 1
   if (retval < 0)
     {
-      builtin_error (_("read error: %d: %s"), fd, strerror (errno));
+      t_errno = errno;
+      if (errno != EINTR)
+       builtin_error (_("read error: %d: %s"), fd, strerror (errno));
       run_unwind_frame ("read_builtin");
-      return (EXECUTION_FAILURE);
+      return ((t_errno != EINTR) ? EXECUTION_FAILURE : 128+lastsig);
     }
-#endif
 
   if (tmsec > 0 || tmusec > 0)
-    reset_alarm ();
+    reset_timeout ();
 
   if (nchars > 0 || delim != '\n')
     {
@@ -601,34 +856,30 @@ add_char:
   if (unbuffered_read == 0)
     zsyncfd (fd);
 
+#if defined (READLINE)
+  if (save_instream)
+    rl_instream = save_instream;       /* can't portably free it */
+#endif
+
   discard_unwind_frame ("read_builtin");
 
   retval = eof ? EXECUTION_FAILURE : EXECUTION_SUCCESS;
 
 assign_vars:
 
-  interrupt_immediately--;
-  terminate_immediately--;
-
 #if defined (ARRAY_VARS)
   /* If -a was given, take the string read, break it into a list of words,
      an assign them to `arrayname' in turn. */
   if (arrayname)
     {
-      if (legal_identifier (arrayname) == 0)
-       {
-         sh_invalidid (arrayname);
-         xfree (input_string);
-         return (EXECUTION_FAILURE);
-       }
-
-      var = find_or_make_array_variable (arrayname, 1);
+      /* pass 1 for flags arg to clear the existing array + 2 to check for a
+        valid identifier. */
+      var = builtin_find_indexed_array (arrayname, 3);
       if (var == 0)
        {
-         xfree (input_string);
+         free (input_string);
          return EXECUTION_FAILURE;     /* readonly or noassign */
        }
-      array_flush (array_cell (var));
 
       alist = list_string (input_string, ifs_chars, 0);
       if (alist)
@@ -640,7 +891,7 @@ assign_vars:
          assign_array_var_from_word_list (var, alist, 0);
          dispose_words (alist);
        }
-      xfree (input_string);
+      free (input_string);
       return (retval);
     }
 #endif /* ARRAY_VARS */ 
@@ -671,7 +922,10 @@ assign_vars:
        }
       else
        var = bind_variable ("REPLY", input_string, 0);
-      VUNSETATTR (var, att_invisible);
+      if (var == 0 || readonly_p (var) || noassign_p (var))
+       retval = EXECUTION_FAILURE;
+      else
+       VUNSETATTR (var, att_invisible);
 
       free (input_string);
       return (retval);
@@ -690,13 +944,14 @@ assign_vars:
     {
       varname = list->word->word;
 #if defined (ARRAY_VARS)
-      if (legal_identifier (varname) == 0 && valid_array_reference (varname) == 0)
+      SET_VFLAGS (list->word->flags, vflags, bindflags);
+      if (legal_identifier (varname) == 0 && valid_array_reference (varname, vflags) == 0)
 #else
       if (legal_identifier (varname) == 0)
 #endif
        {
          sh_invalidid (varname);
-         xfree (orig_input_string);
+         free (orig_input_string);
          return (EXECUTION_FAILURE);
        }
 
@@ -713,22 +968,22 @@ assign_vars:
          if (t && saw_escape)
            {
              t1 = dequote_string (t);
-             var = bind_read_variable (varname, t1);
-             xfree (t1);
+             var = bind_read_variable (varname, t1, bindflags);
+             free (t1);
            }
          else
-           var = bind_read_variable (varname, t);
+           var = bind_read_variable (varname, t ? t : "", bindflags);
        }
       else
        {
          t = (char *)0;
-         var = bind_read_variable (varname, "");
+         var = bind_read_variable (varname, "", bindflags);
        }
 
       FREE (t);
       if (var == 0)
        {
-         xfree (orig_input_string);
+         free (orig_input_string);
          return (EXECUTION_FAILURE);
        }
 
@@ -738,13 +993,14 @@ assign_vars:
 
   /* Now assign the rest of the line to the last variable argument. */
 #if defined (ARRAY_VARS)
-  if (legal_identifier (list->word->word) == 0 && valid_array_reference (list->word->word) == 0)
+  SET_VFLAGS (list->word->flags, vflags, bindflags);
+  if (legal_identifier (list->word->word) == 0 && valid_array_reference (list->word->word, vflags) == 0)
 #else
   if (legal_identifier (list->word->word) == 0)
 #endif
     {
       sh_invalidid (list->word->word);
-      xfree (orig_input_string);
+      free (orig_input_string);
       return (EXECUTION_FAILURE);
     }
 
@@ -771,36 +1027,39 @@ assign_vars:
     }
 #endif
 
-  if (saw_escape)
+  if (saw_escape && input_string && *input_string)
     {
       t = dequote_string (input_string);
-      var = bind_read_variable (list->word->word, t);
-      xfree (t);
+      var = bind_read_variable (list->word->word, t, bindflags);
+      free (t);
     }
   else
-    var = bind_read_variable (list->word->word, input_string);
-  stupidly_hack_special_variables (list->word->word);
-  FREE (tofree);
+    var = bind_read_variable (list->word->word, input_string ? input_string : "", bindflags);
 
   if (var)
-    VUNSETATTR (var, att_invisible);
-  xfree (orig_input_string);
+    {
+      stupidly_hack_special_variables (list->word->word);
+      VUNSETATTR (var, att_invisible);
+    }
+  else
+    retval = EXECUTION_FAILURE;
+
+  FREE (tofree);
+  free (orig_input_string);
 
   return (retval);
 }
 
 static SHELL_VAR *
-bind_read_variable (name, value)
+bind_read_variable (name, value, flags)
      char *name, *value;
+     int flags;
 {
-#if defined (ARRAY_VARS)
-  if (valid_array_reference (name) == 0)
-    return (bind_variable (name, value, 0));
-  else
-    return (assign_array_element (name, value, 0));
-#else /* !ARRAY_VARS */
-  return bind_variable (name, value, 0);
-#endif /* !ARRAY_VARS */
+  SHELL_VAR *v;
+
+  v = builtin_bind_variable (name, value, flags);
+  return (v == 0 ? v
+                : ((readonly_p (v) || noassign_p (v)) ? (SHELL_VAR *)NULL : v));
 }
 
 #if defined (HANDLE_MULTIBYTE)
@@ -829,11 +1088,15 @@ read_mbchar (fd, string, ind, ch, unbuffered)
       if (ret == (size_t)-2)
        {
          ps = ps_back;
-         if (unbuffered)
+
+         /* We don't want to be interrupted during a multibyte char read */
+         if (unbuffered == 2)
+           r = zreadn (fd, &c, 1);
+         else if (unbuffered)
            r = zread (fd, &c, 1);
          else
            r = zreadc (fd, &c);
-         if (r < 0)
+         if (r <= 0)
            goto mbchar_return;
          mbchar[i++] = c;      
          continue;
@@ -856,7 +1119,21 @@ static void
 ttyrestore (ttp)
      struct ttsave *ttp;
 {
-  ttsetattr (ttp->fd, ttp->attrs);
+  ttsetattr (ttp->fd, &(ttp->attrs));
+  tty_modified = 0;
+}
+
+void
+read_tty_cleanup ()
+{
+  if (tty_modified)
+    ttyrestore (&termsave);
+}
+
+int
+read_tty_modified ()
+{
+  return (tty_modified);
 }
 
 #if defined (READLINE)
@@ -903,18 +1180,30 @@ edit_line (p, itext)
 
   old_attempted_completion_function = rl_attempted_completion_function;
   rl_attempted_completion_function = (rl_completion_func_t *)NULL;
+  bashline_set_event_hook ();
   if (itext)
     {
       old_startup_hook = rl_startup_hook;
       rl_startup_hook = set_itext;
       deftext = itext;
     }
+
   ret = readline (p);
+
   rl_attempted_completion_function = old_attempted_completion_function;
   old_attempted_completion_function = (rl_completion_func_t *)NULL;
+  bashline_reset_event_hook ();
 
   if (ret == 0)
-    return ret;
+    {
+      if (RL_ISSTATE (RL_STATE_TIMEOUT))
+       {
+         sigalrm (SIGALRM);            /* simulate receiving SIGALRM */
+         check_read_timeout ();
+       }
+      return ret;
+    }
+
   len = strlen (ret);
   ret = (char *)xrealloc (ret, len + 2);
   ret[len++] = delim;
@@ -922,6 +1211,17 @@ edit_line (p, itext)
   return ret;
 }
 
+static void
+set_readline_timeout (t, sec, usec)
+     sh_timer *t;
+     time_t sec;
+     long usec;
+{
+  t->tmout.tv_sec = sec;
+  t->tmout.tv_usec = usec;
+  rl_set_timeout (sec, usec);
+}
+
 static int old_delim_ctype;
 static rl_command_func_t *old_delim_func;
 static int old_newline_ctype;
@@ -939,15 +1239,17 @@ set_eol_delim (c)
     initialize_readline ();
   cmap = rl_get_keymap ();
 
-  /* Change newline to self-insert */
+  /* Save the old delimiter char binding */
   old_newline_ctype = cmap[RETURN].type;
   old_newline_func =  cmap[RETURN].function;
+  old_delim_ctype = cmap[c].type;
+  old_delim_func = cmap[c].function;
+
+  /* Change newline to self-insert */
   cmap[RETURN].type = ISFUNC;
   cmap[RETURN].function = rl_insert;
 
   /* Bind the delimiter character to accept-line. */
-  old_delim_ctype = cmap[c].type;
-  old_delim_func = cmap[c].function;
   cmap[c].type = ISFUNC;
   cmap[c].function = rl_newline;