aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRob Landley <rob@landley.net>2019-07-31 11:28:26 -0500
committerRob Landley <rob@landley.net>2019-07-31 11:28:26 -0500
commit88468a07124a5cda235037249a65a32c4c95f1dd (patch)
tree62759ebbdfa32277c7c30752a5df4a1591f7bdae
parent65ac89cd4605ba01a6154c903fd54e2f9f7e6147 (diff)
downloadtoybox-88468a07124a5cda235037249a65a32c4c95f1dd.tar.gz
Fix error path and tweak comments.
-rw-r--r--lib/xwrap.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/lib/xwrap.c b/lib/xwrap.c
index 817037db..1e39c1bc 100644
--- a/lib/xwrap.c
+++ b/lib/xwrap.c
@@ -197,7 +197,7 @@ pid_t __attribute__((returns_twice)) xvforkwrap(pid_t pid)
if (pid == -1) perror_exit("vfork");
// Signal to xexec() and friends that we vforked so can't recurse
- toys.stacktop = 0;
+ if (!pid) toys.stacktop = 0;
return pid;
}
@@ -237,24 +237,24 @@ pid_t xpopen_both(char **argv, int *pipes)
if (!(pid = CFG_TOYBOX_FORK ? xfork() : XVFORK())) {
// Child process: Dance of the stdin/stdout redirection.
- // cestnepasun[1]->cestnepasun[0] and cestnepasun[2]->cestnepasun[1]
+ // cestnepasun[1]->cestnepasun[0] and cestnepasun[3]->cestnepasun[2]
if (pipes) {
// if we had no stdin/out, pipe handles could overlap, so test for it
// and free up potentially overlapping pipe handles before reuse
- // in child, close read end of output pipe, and return write end as out
+ // in child, close read end of output pipe, use write end as new stdout
if (cestnepasun[2]) {
close(cestnepasun[2]);
pipes[1] = cestnepasun[3];
}
- // in child, close write end of input pipe, and return input end as out.
+ // in child, close write end of input pipe, use read end as new stdin
if (cestnepasun[1]) {
close(cestnepasun[1]);
pipes[0] = cestnepasun[0];
}
- // If swapping stdin/stdout, need to move a filehandle to temp
+ // If swapping stdin/stdout, dup a filehandle that gets closed before use
if (!pipes[1]) pipes[1] = dup(0);
// Are we redirecting stdin?