From 68286770b61638b99e110051612a6ee27088b1b4 Mon Sep 17 00:00:00 2001 From: Rob Landley Date: Tue, 13 Jun 2006 19:01:30 +0000 Subject: Patch from Shaun Jackman to detect "name=value" variable assignments. Random cleanup at the end by me to make some unrelated #ifdefs go away... --- shell/lash.c | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) (limited to 'shell') diff --git a/shell/lash.c b/shell/lash.c index 216db8a7e..eebb2f8a2 100644 --- a/shell/lash.c +++ b/shell/lash.c @@ -1182,9 +1182,12 @@ static int parse_command(char **command_ptr, struct job *job, int *inbg) static int pseudo_exec(struct child_prog *child) { struct built_in_command *x; -#ifdef CONFIG_FEATURE_SH_STANDALONE_SHELL - char *name; -#endif + + /* Check if the command sets an environment variable. */ + if( strchr(child->argv[0], '=') != NULL ) { + child->argv[1] = child->argv[0]; + _exit(builtin_export(child)); + } /* Check if the command matches any of the non-forking builtins. * Depending on context, this might be redundant. But it's @@ -1204,7 +1207,7 @@ static int pseudo_exec(struct child_prog *child) _exit (x->function(child)); } } -#ifdef CONFIG_FEATURE_SH_STANDALONE_SHELL + /* Check if the command matches any busybox internal * commands ("applets") here. Following discussions from * November 2000 on busybox@busybox.net, don't use @@ -1216,17 +1219,15 @@ static int pseudo_exec(struct child_prog *child) * /bin/foo invocation will fork and exec /bin/foo, even if * /bin/foo is a symlink to busybox. */ - name = child->argv[0]; - { - char** argv_l=child->argv; + if (ENABLE_FEATURE_SH_STANDALONE_SHELL) { + char **argv_l = child->argv; int argc_l; - for(argc_l=0;*argv_l!=NULL; argv_l++, argc_l++); + for(argc_l=0; *argv_l; argv_l++, argc_l++); optind = 1; - run_applet_by_name(name, argc_l, child->argv); + run_applet_by_name(child->argv[0], argc_l, child->argv); } -#endif execvp(child->argv[0], child->argv); -- cgit v1.2.3