diff options
| author | Denys Vlasenko | 2016-10-26 16:26:45 +0200 |
|---|---|---|
| committer | Denys Vlasenko | 2016-10-26 16:26:45 +0200 |
| commit | 350e686f3b04f41f623316706094f0e18a10c1cf (patch) | |
| tree | 721ff03c6f5538060cc91f5c03bb9047c354970c /shell/ash.c | |
| parent | f15aa57a7f5edcbf3098873b8798c0ea7f496ed7 (diff) | |
| download | busybox-350e686f3b04f41f623316706094f0e18a10c1cf.zip busybox-350e686f3b04f41f623316706094f0e18a10c1cf.tar.gz | |
ash: [PARSER] Recognise here-doc delimiters terminated by EOF
Upstream commit 1:
Date: Wed, 26 Sep 2007 17:14:16 +0800
[PARSER] Recognise here-doc delimiters terminated by EOF
Previously dash required a <newline> character to be present in order for
a here-document delimiter to be detected. Allowing EOF in the absence of
a <newline> to play the same purpose allows some intuitive scripts to
succeed. POSIX seems to be silence on this so this should be OK.
Test case:
eval 'cat <<- NOT
test
NOT'
echo OK
Old result:
test
NOTOK
New result:
test
OK
Upstream commit 2:
Date: Sat, 20 Oct 2007 18:49:31 +0800
[PARSER] Fix here-doc corruption
The change
[PARSER] Recognise here-doc delimiters terminated by EOF
introduced a regerssion whereby lines starting with eofmark but are not equal
to eofmark would be corrupted. This patch fixes it.
Test case:
cat << _ACEOF
_ASBOX
_ACEOF
Old result:
SASBOX
New result:
_ASBOX
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'shell/ash.c')
| -rw-r--r-- | shell/ash.c | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/shell/ash.c b/shell/ash.c index 2cebfe2..e0828d4 100644 --- a/shell/ash.c +++ b/shell/ash.c @@ -11592,11 +11592,17 @@ checkend: { if (c == *eofmark) { if (pfgets(line, sizeof(line)) != NULL) { char *p, *q; + int cc; p = line; - for (q = eofmark + 1; *q && *p == *q; p++, q++) - continue; - if (*p == '\n' && *q == '\0') { + for (q = eofmark + 1;; p++, q++) { + cc = *p; + if (cc == '\n') + cc = 0; + if (!*q || cc != *q) + break; + } + if (cc == *q) { c = PEOF; nlnoprompt(); } else { |