One more case, which currently segfaults:

declare -n REPLY; read <<< /

easy fix:

diff --git a/builtins/read.def b/builtins/read.def
index 5e2348c..8f6bd2b 100644--- a/builtins/read.def+++ b/builtins/read.def
@@ -797,9 +797,11 @@ assign_vars:
        }
       else
        var = bind_variable ("REPLY", input_string, 0);-
VUNSETATTR (var, att_invisible);-      if (readonly_p (var) ||
noassign_p (var))++      if (var == 0 || readonly_p (var) ||
noassign_p (var))
        retval = EXECUTION_FAILURE;+      else+        VUNSETATTR
(var, att_invisible);

       xfree (input_string);
       return (retval);

​

On Fri, May 6, 2016 at 11:39 AM, Chet Ramey <chet.ra...@case.edu> wrote:

> On 4/27/16 4:07 PM, Eduardo A. Bustamante López wrote:
> > Here's the updated list of cases:
>
> Thanks for the list of test cases; these were very helpful.
>
> Chet
>
> --
> ``The lyf so short, the craft so long to lerne.'' - Chaucer
>                  ``Ars longa, vita brevis'' - Hippocrates
> Chet Ramey, ITS, CWRU    c...@case.edu
> http://cnswww.cns.cwru.edu/~chet/
>

Reply via email to