diff options
author | peter <peter@FreeBSD.org> | 1996-09-10 01:24:11 +0000 |
---|---|---|
committer | peter <peter@FreeBSD.org> | 1996-09-10 01:24:11 +0000 |
commit | aae29b5fef43167b0ab23b5350b23a6312867900 (patch) | |
tree | 34149b17c1927c43b2dd2f20f5d9a5cf6cdd4c56 /bin/sh/expand.c | |
parent | c5a6fcbc6ced2de2faae882dbb27a2694688937b (diff) | |
download | FreeBSD-src-aae29b5fef43167b0ab23b5350b23a6312867900.zip FreeBSD-src-aae29b5fef43167b0ab23b5350b23a6312867900.tar.gz |
Fix for PR#1248, sh doesn't expand past ${9}
Submitted by: Steve Price <sprice@hiwaay.net>
Diffstat (limited to 'bin/sh/expand.c')
-rw-r--r-- | bin/sh/expand.c | 36 |
1 files changed, 20 insertions, 16 deletions
diff --git a/bin/sh/expand.c b/bin/sh/expand.c index 99bbb3c..ee88ad8 100644 --- a/bin/sh/expand.c +++ b/bin/sh/expand.c @@ -33,7 +33,7 @@ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. * - * $Id: expand.c,v 1.6 1996/08/12 19:31:11 ache Exp $ + * $Id: expand.c,v 1.7 1996/09/01 10:20:07 peter Exp $ */ #ifndef lint @@ -97,8 +97,8 @@ STATIC char *exptilde __P((char *, int)); STATIC void expbackq __P((union node *, int, int)); STATIC int subevalvar __P((char *, char *, int, int, int)); STATIC char *evalvar __P((char *, int)); -STATIC int varisset __P((int)); -STATIC void varvalue __P((int, int, int)); +STATIC int varisset __P((char *)); +STATIC void varvalue __P((char *, int, int)); STATIC void recordregion __P((int, int, int)); STATIC void ifsbreakup __P((char *, struct arglist *)); STATIC void expandmeta __P((struct strlist *, int)); @@ -547,7 +547,7 @@ evalvar(p, flag) p = strchr(p, '=') + 1; again: /* jump here after setting a variable with ${var=text} */ if (special) { - set = varisset(*var); + set = varisset(var); val = NULL; } else { val = lookupvar(var); @@ -563,7 +563,7 @@ again: /* jump here after setting a variable with ${var=text} */ /* insert the value of the variable */ if (special) { char *exp, *oexpdest = expdest; - varvalue(*var, varflags & VSQUOTE, flag & EXP_FULL); + varvalue(var, varflags & VSQUOTE, flag & EXP_FULL); if (subtype == VSLENGTH) { for (exp = oexpdest;exp != expdest; exp++) varlen++; @@ -677,22 +677,23 @@ record: STATIC int varisset(name) - char name; + char *name; { char **ap; + int num; - if (name == '!') { + if (*name == '!') { if (backgndpid == -1) return 0; - } else if (name == '@' || name == '*') { + } else if (*name == '@' || *name == '*') { if (*shellparam.p == NULL) return 0; - } else if ((unsigned)(name -= '1') <= '9' - '1') { + } else if (is_digit(*name)) { + num = atoi(name); ap = shellparam.p; - do { + while (num-- > 0) if (*ap++ == NULL) return 0; - } while (--name >= 0); } return 1; } @@ -705,7 +706,7 @@ varisset(name) STATIC void varvalue(name, quoted, allow_split) - char name; + char *name; int quoted; int allow_split; { @@ -732,7 +733,7 @@ varvalue(name, quoted, allow_split) } while (0) - switch (name) { + switch (*name) { case '$': num = rootpid; goto numvar; @@ -773,9 +774,12 @@ allargs: STRTODEST(p); break; default: - if ((unsigned)(name -= '1') <= '9' - '1') { - p = shellparam.p[name]; - STRTODEST(p); + if (is_digit(*name)) { + num = atoi(name); + if (num > 0 && num <= shellparam.nparam) { + p = shellparam.p[*name]; + STRTODEST(p); + } } break; } |