From: Matt Kraai Date: Wed, 6 Sep 2000 01:46:18 +0000 (-0000) Subject: Reset optind so that getopt works in busybox applets called directly by sh. X-Git-Tag: 0_47~111 X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=6085c72b6ee92d69642a85b6f5d2dff239c5b1c1;p=oweals%2Fbusybox.git Reset optind so that getopt works in busybox applets called directly by sh. Change the behavior of -c to correctly handle $1..$9. --- diff --git a/lash.c b/lash.c index 50e4368db..5a2547368 100644 --- a/lash.c +++ b/lash.c @@ -1193,6 +1193,7 @@ static int runCommand(struct job *newJob, struct jobSet *jobList, int inBg, int char** argv=newJob->progs[i].argv; for(argc_l=0;*argv!=NULL; argv++, argc_l++); applet_name=a->name; + optind = 1; exit((*(a->main)) (argc_l, newJob->progs[i].argv)); } a++; @@ -1393,26 +1394,15 @@ int shell_main(int argc_l, char **argv_l) // builtin_source("/etc/profile"); //} - while ((opt = getopt(argc, argv, "cx")) > 0) { + while ((opt = getopt(argc_l, argv_l, "cx")) > 0) { switch (opt) { case 'c': input = NULL; - local_pending_command = (char *) calloc(BUFSIZ, sizeof(char)); - if (local_pending_command == 0) { - fatalError("sh: out of memory\n"); - } - for(; optind= BUFSIZ) { - local_pending_command = realloc(local_pending_command, - strlen(local_pending_command) + strlen(argv[optind])); - if (local_pending_command==NULL) - fatalError("command too long\n"); - } - strcat(local_pending_command, argv[optind]); - if ( (optind + 1) < argc) - strcat(local_pending_command, " "); - } + if (local_pending_command != 0) + fatalError("multiple -c arguments\n"); + local_pending_command = xstrdup(argv[optind]); + optind++; + argv = argv+optind; break; #ifdef BB_FEATURE_SH_ENVIRONMENT case 'x': diff --git a/sh.c b/sh.c index 50e4368db..5a2547368 100644 --- a/sh.c +++ b/sh.c @@ -1193,6 +1193,7 @@ static int runCommand(struct job *newJob, struct jobSet *jobList, int inBg, int char** argv=newJob->progs[i].argv; for(argc_l=0;*argv!=NULL; argv++, argc_l++); applet_name=a->name; + optind = 1; exit((*(a->main)) (argc_l, newJob->progs[i].argv)); } a++; @@ -1393,26 +1394,15 @@ int shell_main(int argc_l, char **argv_l) // builtin_source("/etc/profile"); //} - while ((opt = getopt(argc, argv, "cx")) > 0) { + while ((opt = getopt(argc_l, argv_l, "cx")) > 0) { switch (opt) { case 'c': input = NULL; - local_pending_command = (char *) calloc(BUFSIZ, sizeof(char)); - if (local_pending_command == 0) { - fatalError("sh: out of memory\n"); - } - for(; optind= BUFSIZ) { - local_pending_command = realloc(local_pending_command, - strlen(local_pending_command) + strlen(argv[optind])); - if (local_pending_command==NULL) - fatalError("command too long\n"); - } - strcat(local_pending_command, argv[optind]); - if ( (optind + 1) < argc) - strcat(local_pending_command, " "); - } + if (local_pending_command != 0) + fatalError("multiple -c arguments\n"); + local_pending_command = xstrdup(argv[optind]); + optind++; + argv = argv+optind; break; #ifdef BB_FEATURE_SH_ENVIRONMENT case 'x': diff --git a/shell/lash.c b/shell/lash.c index 50e4368db..5a2547368 100644 --- a/shell/lash.c +++ b/shell/lash.c @@ -1193,6 +1193,7 @@ static int runCommand(struct job *newJob, struct jobSet *jobList, int inBg, int char** argv=newJob->progs[i].argv; for(argc_l=0;*argv!=NULL; argv++, argc_l++); applet_name=a->name; + optind = 1; exit((*(a->main)) (argc_l, newJob->progs[i].argv)); } a++; @@ -1393,26 +1394,15 @@ int shell_main(int argc_l, char **argv_l) // builtin_source("/etc/profile"); //} - while ((opt = getopt(argc, argv, "cx")) > 0) { + while ((opt = getopt(argc_l, argv_l, "cx")) > 0) { switch (opt) { case 'c': input = NULL; - local_pending_command = (char *) calloc(BUFSIZ, sizeof(char)); - if (local_pending_command == 0) { - fatalError("sh: out of memory\n"); - } - for(; optind= BUFSIZ) { - local_pending_command = realloc(local_pending_command, - strlen(local_pending_command) + strlen(argv[optind])); - if (local_pending_command==NULL) - fatalError("command too long\n"); - } - strcat(local_pending_command, argv[optind]); - if ( (optind + 1) < argc) - strcat(local_pending_command, " "); - } + if (local_pending_command != 0) + fatalError("multiple -c arguments\n"); + local_pending_command = xstrdup(argv[optind]); + optind++; + argv = argv+optind; break; #ifdef BB_FEATURE_SH_ENVIRONMENT case 'x':