From a817a02c00a06fbf2090259320968addb5f46e58 Mon Sep 17 00:00:00 2001
From: Rob Landley <rob@landley.net>
Date: Thu, 18 Jul 2013 18:20:03 -0500
Subject: Replace users of xexec(toys.optargs) with xexec_optargs(0) to avoid
 free/reuse bug during argument parsing.

---
 toys/other/chroot.c  | 4 +++-
 toys/other/netcat.c  | 2 +-
 toys/other/setsid.c  | 2 +-
 toys/other/taskset.c | 2 +-
 toys/other/unshare.c | 2 +-
 5 files changed, 7 insertions(+), 5 deletions(-)

(limited to 'toys/other')

diff --git a/toys/other/chroot.c b/toys/other/chroot.c
index c9ff10c7..00fb4e7f 100644
--- a/toys/other/chroot.c
+++ b/toys/other/chroot.c
@@ -18,6 +18,8 @@ config CHROOT
 void chroot_main(void)
 {
   char *binsh[] = {"/bin/sh", "-i", 0};
+
   if (chdir(*toys.optargs) || chroot(".")) perror_exit("%s", *toys.optargs);
-  xexec(toys.optargs[1] ? toys.optargs+1 : binsh);
+  if (toys.optargs[1]) xexec_optargs(1);
+  else xexec(binsh);
 }
diff --git a/toys/other/netcat.c b/toys/other/netcat.c
index a245d119..e41fc4b2 100644
--- a/toys/other/netcat.c
+++ b/toys/other/netcat.c
@@ -187,7 +187,7 @@ void netcat_main(void)
   set_alarm(0);
 
   if (CFG_NETCAT_LISTEN && (toys.optflags&(FLAG_L|FLAG_l) && toys.optc))
-    xexec(toys.optargs);
+    xexec_optargs(0);
 
   // Poll loop copying stdin->socket and socket->stdout.
   for (;;) {
diff --git a/toys/other/setsid.c b/toys/other/setsid.c
index 59a1d78f..8f0a0647 100644
--- a/toys/other/setsid.c
+++ b/toys/other/setsid.c
@@ -24,5 +24,5 @@ void setsid_main(void)
     setpgid(0,0);
     tcsetpgrp(0, getpid());
   }
-  xexec(toys.optargs);
+  xexec_optargs(0);
 }
diff --git a/toys/other/taskset.c b/toys/other/taskset.c
index 2b067d4a..bcc03478 100644
--- a/toys/other/taskset.c
+++ b/toys/other/taskset.c
@@ -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(toys.optargs+1);
+    xexec_optargs(1);
   } else {
     char *c;
     pid_t pid = strtol(toys.optargs[toys.optc-1], &c, 10);
diff --git a/toys/other/unshare.c b/toys/other/unshare.c
index 9a938186..87db6039 100644
--- a/toys/other/unshare.c
+++ b/toys/other/unshare.c
@@ -35,5 +35,5 @@ void unshare_main(void)
 
   if(unshare(f)) perror_exit("failed");
 
-  xexec(toys.optargs);
+  xexec_optargs(0);
 }
-- 
cgit v1.2.3