Re: Alias/heredoc/cmdsubst regression

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 24/01/2020 23:48, Martijn Dekker wrote:
There is a regression involving alias expansion, a here-document, and a command substitution. The following worked fine until dash 0.5.8; it throws a syntax error as of dash 0.5.9.

alias BEGIN='{' END='}'
BEGIN
cat <<eof
$(echo hi)
eof
END

Nice find.

When the newline after cat <<eof is seen, checkkwd is changed to indicate that the shell is in a state where aliases can be expanded. Then, parseheredoc() is called, which in turn calls readtoken1() to parse the here-document. readtoken() re-sets checkkwd once it is done, but readtoken1() does not, so normally this preserves the "can expand aliases" state. However, nested command substitutions do reset checkkwd, so things break.

Until 0.5.8, parseheredoc() was called first, and only after that did checkkwd get changed.

Either parseheredoc() needs to save and restore checkkwd, or the code calling parseheredoc() needs to ensure that it sets checkkwd as appropriate afterwards.

Cheers,
Harald van Dijk



[Index of Archives]     [LARTC]     [Bugtraq]     [Yosemite Forum]     [Photo]

  Powered by Linux