Make toy_exec() check if argc is in optargs and deal with it there so we don't need...
authorRob Landley <rob@landley.net>
Sat, 7 Feb 2015 22:17:44 +0000 (16:17 -0600)
committerRob Landley <rob@landley.net>
Sat, 7 Feb 2015 22:17:44 +0000 (16:17 -0600)
12 files changed:
main.c
toys/other/chroot.c
toys/other/netcat.c
toys/other/nsenter.c
toys/other/setsid.c
toys/other/taskset.c
toys/other/timeout.c
toys/pending/bootchartd.c
toys/pending/tcpsvd.c
toys/posix/nice.c
toys/posix/nohup.c
toys/posix/time.c

diff --git a/main.c b/main.c
index 7f74549..458d470 100644 (file)
--- a/main.c
+++ b/main.c
@@ -127,6 +127,9 @@ void toy_exec(char *argv[])
 {
   struct toy_list *which;
 
+  // don't blank old optargs if our new argc lives in the old optargs.
+  if (argv>=toys.optargs && argv<=toys.optargs+toys.optc) toys.optargs = 0;
+
   // Return if we can't find it, or need to re-exec to acquire root,
   // or if stack depth is getting silly.
   if (!(which = toy_find(argv[0]))) return;
index 00fb4e7..e82dd80 100644 (file)
@@ -20,6 +20,6 @@ void chroot_main(void)
   char *binsh[] = {"/bin/sh", "-i", 0};
 
   if (chdir(*toys.optargs) || chroot(".")) perror_exit("%s", *toys.optargs);
-  if (toys.optargs[1]) xexec_optargs(1);
+  if (toys.optargs[1]) xexec(toys.optargs+1);
   else xexec(binsh);
 }
index 616822b..d27aa88 100644 (file)
@@ -185,7 +185,7 @@ void netcat_main(void)
   set_alarm(0);
 
   if (CFG_NETCAT_LISTEN && (toys.optflags&(FLAG_L|FLAG_l) && toys.optc))
-    xexec_optargs(0);
+    xexec(toys.optargs);
 
   // Poll loop copying stdin->socket and socket->stdout.
   for (;;) {
index d53de41..8f548bf 100644 (file)
@@ -112,5 +112,5 @@ void unshare_main(void)
     }
   }
 
-  xexec_optargs(0);
+  xexec(toys.optargs);
 }
index 8f0a064..59a1d78 100644 (file)
@@ -24,5 +24,5 @@ void setsid_main(void)
     setpgid(0,0);
     tcsetpgrp(0, getpid());
   }
-  xexec_optargs(0);
+  xexec(toys.optargs);
 }
index bcc0347..2b067d4 100644 (file)
@@ -90,7 +90,7 @@ void taskset_main(void)
   if (!(toys.optflags & FLAG_p)) {
     if (toys.optc < 2) error_exit("Needs 2 args");
     do_taskset(getpid(), 1);
-    xexec_optargs(1);
+    xexec(toys.optargs+1);
   } else {
     char *c;
     pid_t pid = strtol(toys.optargs[toys.optc-1], &c, 10);
index f8acabf..3824376 100644 (file)
@@ -60,7 +60,7 @@ void timeout_main(void)
   if (TT.s_signal && -1 == (TT.nextsig = sig_to_num(TT.s_signal)))
     error_exit("bad -s: '%s'", TT.s_signal);
 
-  if (!(TT.pid = xfork())) xexec_optargs(1);
+  if (!(TT.pid = xfork())) xexec(toys.optargs+1);
   else {
     int status;
 
index b29d4f0..d903405 100644 (file)
@@ -309,7 +309,7 @@ void bootchartd_main()
   if (bchartd_opt == 1 && toys.optargs[1]) { 
     pid_t prog_pid;
 
-    if (!(prog_pid = xfork())) xexec_optargs(1);
+    if (!(prog_pid = xfork())) xexec(toys.optargs+1);
     waitpid(prog_pid, NULL, 0);
     kill(lgr_pid, SIGUSR1);
   }
index 585ae73..16110e5 100644 (file)
@@ -392,7 +392,7 @@ void tcpsvd_main(void)
       close(1);
       dup2(newfd, 0);
       dup2(newfd, 1);
-      xexec_optargs(2); //skip IP PORT
+      xexec(toys.optargs+2); //skip IP PORT
     } else {
       insert(&pids, pid, addr);
       xclose(newfd); //close and reopen for next client.
index bc25d35..4b587ee 100644 (file)
@@ -34,5 +34,5 @@ void nice_main(void)
   errno = 0;
   if (nice(TT.priority)==-1 && errno) perror_exit("Can't set priority");
 
-  xexec_optargs(0);
+  xexec(toys.optargs);
 }
index df264da..658ab93 100644 (file)
@@ -38,5 +38,5 @@ void nohup_main(void)
     close(0);
     open("/dev/null", O_RDONLY);
   }
-  xexec_optargs(0);
+  xexec(toys.optargs);
 }
index 0159205..01da971 100644 (file)
@@ -27,7 +27,7 @@ void time_main(void)
   struct timeval tv, tv2;
 
   gettimeofday(&tv, NULL);
-  if (!(pid = xfork())) xexec_optargs(0);
+  if (!(pid = xfork())) xexec(toys.optargs);
   else {
     int stat;
     struct rusage ru;