On 4/9/17 6:12 PM, Grisha Levit wrote:
> On Sun, Apr 9, 2017 at 5:56 PM, Chet Ramey <chet.ra...@case.edu> wrote:
>> Thanks for the report and fix.  This is the patch where you could have used
>> dequote_escapes() ;-) .
> 
> Yup, I realized I'd done the previous one incorrectly shortly after writing
> this one, sorry about that.
> 
> But I'm not sure about dequoting the function name, so I wanted to dequote
> just the value, not the whole sting pointed to by q.  In retrospect, maybe
> I should have clarified that issue first:
> 
> $ eval $'\001() { :; }'
> $ declare -F | xxd
> 00000000: 6465 636c 6172 6520 2d66 2001 010a       declare -f ...
>                                      ^^ ^^
> 
> The actual function that's created actually *is* $'\001\001':

No, that's true, and it should be fixed.  At some point, though, you have
to ask yourself what someone would be doing creating a function named ^A.

Chet

-- 
``The lyf so short, the craft so long to lerne.'' - Chaucer
                 ``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, UTech, CWRU    c...@case.edu    http://cnswww.cns.cwru.edu/~chet/

Reply via email to