summaryrefslogtreecommitdiff
path: root/lash.c
diff options
context:
space:
mode:
authorEric Andersen2001-03-09 20:34:46 +0000
committerEric Andersen2001-03-09 20:34:46 +0000
commit74e056bfc8e2d6f006548c0134502697c056f97a (patch)
tree676e4afbfb2edbbbe734d80b05c19706c4fdb37b /lash.c
parent3c7361f53aab0d4c6cb3c46218810bda52693fab (diff)
downloadbusybox-74e056bfc8e2d6f006548c0134502697c056f97a.zip
busybox-74e056bfc8e2d6f006548c0134502697c056f97a.tar.gz
Fixed a couple more cases. $FOO/bar ${FOO} and such now work
without wordexp. Of course for stuff like ${1:-foo} you still need wordexp for them to work. -Erik
Diffstat (limited to 'lash.c')
-rw-r--r--lash.c18
1 files changed, 14 insertions, 4 deletions
diff --git a/lash.c b/lash.c
index 34dca04..8f19e52 100644
--- a/lash.c
+++ b/lash.c
@@ -1014,7 +1014,6 @@ static int expand_arguments(char *command)
* wordexp can't do for us, namely $? and $! */
src = command;
while((dst = strchr(src,'$')) != NULL){
- printf("dollar '%s'\n", dst);
var = NULL;
switch(*(dst+1)) {
case '?':
@@ -1057,14 +1056,25 @@ static int expand_arguments(char *command)
} else {
/* Looks like an environment variable */
char delim_hold;
- src=strpbrk(dst+1, " \t~`!$^&*()=|\\{}[];\"'<>?.");
+ int num_skip_chars=1;
+ int dstlen = strlen(dst);
+ /* Is this a ${foo} type variable? */
+ if (dstlen >=2 && *(dst+1) == '{') {
+ src=strchr(dst+1, '}');
+ num_skip_chars=2;
+ } else {
+ src=strpbrk(dst+1, " \t~`!$^&*()=|\\[];\"'<>?./");
+ }
if (src == NULL) {
- src = dst+strlen(dst);
+ src = dst+dstlen;
}
delim_hold=*src;
*src='\0'; /* temporary */
- var = getenv(dst + 1);
+ var = getenv(dst + num_skip_chars);
*src=delim_hold;
+ if (num_skip_chars==2) {
+ src++;
+ }
}
if (var == NULL) {
/* Seems we got an un-expandable variable. So delete it. */