ash: parser: Do not push token back before parseheredoc
authorDenys Vlasenko <vda.linux@googlemail.com>
Sat, 22 Feb 2020 16:26:23 +0000 (17:26 +0100)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sat, 22 Feb 2020 16:26:23 +0000 (17:26 +0100)
commitc08993f40c0c6c7bdf453e77c3aa9dae8ec0dad9
tree01a44b93192180ac09ba94016f05e9ba8db15fc9
parent9a1a659707a18c29166c3e2977523102866d7aed
ash: parser: Do not push token back before parseheredoc

Upstream commit:

    Date: Mon, 19 Nov 2018 18:43:58 +0800
    parser: Do not push token back before parseheredoc

    When we read the first token in list() we use peektoken instead
    of readtoken as the following code needs to use the same token
    again.  However, this is wrong when we're in a here-document as
    it will clobber the saved token without resetting the tokpushback
    flag.

    This patch fixes it by doing the tokpushback after parseheredoc
    and setting lasttoken again if parseheredoc was called.

Reported-by: Ron Yorston <rmy@frippery.org>
    Fixes: 7c245aa8ed33 ("[PARSER] Simplify EOF/newline handling in...")
    Fixes: ee5cbe9fd6bc ("[SHELL] Optimize dash -c "command" to avoid a fork")
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Tested-by: Simon Ser <contact@emersion.fr>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/ash.c