summary refs log tree commit diff
diff options
context:
space:
mode:
authorHerbert Xu <herbert@gondor.apana.org.au>2019-05-27 13:39:37 +0800
committerHerbert Xu <herbert@gondor.apana.org.au>2020-01-20 17:52:07 +0800
commit7710a926b321879b84bf349b865b1dd4c52083f8 (patch)
tree224c3b41059e522904758d705110de777e19c611
parentexpand: Fix trailing newlines processing in backquote expanding (diff)
downloaddash-7710a926b321879b84bf349b865b1dd4c52083f8.tar.gz
dash-7710a926b321879b84bf349b865b1dd4c52083f8.zip
parser: Only accept single-digit parameter expansion outside of braces
On Thu, Apr 25, 2019 at 01:39:52AM +0000, Michael Orlitzky wrote:
> The POSIX spec says,
>
>   The parameter name or symbol can be enclosed in braces, which are
>   optional except for positional parameters with more than one digit or
>   when parameter is a name and is followed by a character that could be
>   interpreted as part of the name.
>
> However, dash seems to diverge from that behavior when we get to $10:
>
>   $ cat test.sh
>   echo $10
>
>   $ dash ./test.sh one two three four five six seven eight nine ten
>   ten
>
>   $ bash ./test.sh one two three four five six seven eight nine ten
>   one0

This patch should fix the problem.

Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
-rw-r--r--src/parser.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/parser.c b/src/parser.c
index 4bda42e..b318b08 100644
--- a/src/parser.c
+++ b/src/parser.c
@@ -1268,7 +1268,7 @@ varname:
 			do {
 				STPUTC(c, out);
 				c = pgetc_eatbnl();
-			} while (is_digit(c));
+			} while (!subtype && is_digit(c));
 		} else if (c != '}') {
 			int cc = c;