summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko <vda.linux@googlemail.com>2009-09-06 21:48:16 +0200
committerDenys Vlasenko <vda.linux@googlemail.com>2009-09-06 21:48:16 +0200
commitee4233a2f5f47409da9aab5a59c0753e83ef0ab8 (patch)
tree66e9b2f0f8771124997c7a9b7cfeb6b4b833342a
parent656167d89112171b2b7672676dc413c676a186f5 (diff)
wordexp.c: fix a bug where we might close stdout
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--libc/misc/wordexp/wordexp.c13
1 files changed, 8 insertions, 5 deletions
diff --git a/libc/misc/wordexp/wordexp.c b/libc/misc/wordexp/wordexp.c
index 8167ed685..4a2b50b4e 100644
--- a/libc/misc/wordexp/wordexp.c
+++ b/libc/misc/wordexp/wordexp.c
@@ -787,6 +787,7 @@ parse_arith(char **word, size_t * word_length, size_t * max_length,
static void attribute_noreturn
exec_comm_child(char *comm, int *fildes, int showerr, int noexec)
{
+ int fd;
const char *args[4] = { _PATH_BSHELL, "-c", comm, NULL };
/* Execute the command, or just check syntax? */
@@ -794,13 +795,14 @@ exec_comm_child(char *comm, int *fildes, int showerr, int noexec)
args[1] = "-nc";
/* Redirect output. */
- dup2(fildes[1], 1);
- close(fildes[1]);
+ fd = fildes[1];
+ if (fd != 1) {
+ dup2(fd, 1);
+ close(fd);
+ }
/* Redirect stderr to /dev/null if we have to. */
if (showerr == 0) {
- int fd;
-
close(2);
fd = open(_PATH_DEVNULL, O_WRONLY);
if (fd >= 0 && fd != 2) {
@@ -812,7 +814,8 @@ exec_comm_child(char *comm, int *fildes, int showerr, int noexec)
/* Make sure the subshell doesn't field-split on our behalf. */
unsetenv("IFS");
- close(fildes[0]);
+ if (fildes[0] != 1)
+ close(fildes[0]);
execve(_PATH_BSHELL, (char *const *) args, __environ);
/* Bad. What now? */