summaryrefslogtreecommitdiff
path: root/src/process
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2014-12-05 21:15:41 -0500
committerRich Felker <dalias@aerifal.cx>2014-12-05 21:15:41 -0500
commit1c12c24364d1058ffdbb28fca72a51de85082778 (patch)
treec199e4d80a2f97a14e5f310979d8611a57227505 /src/process
parent014275b547e3059db5c45986408757c250e8198d (diff)
downloadmusl-1c12c24364d1058ffdbb28fca72a51de85082778.tar.gz
musl-1c12c24364d1058ffdbb28fca72a51de85082778.tar.bz2
musl-1c12c24364d1058ffdbb28fca72a51de85082778.tar.xz
musl-1c12c24364d1058ffdbb28fca72a51de85082778.zip
don't fail posix_spawn on failed close
the resolution of austin group issue #370 removes the requirement that posix_spawn fail when the close file action is performed on an already-closed fd. since there are no other meaningful errors for close, just ignoring the return value completely is the simplest fix.
Diffstat (limited to 'src/process')
-rw-r--r--src/process/posix_spawn.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/src/process/posix_spawn.c b/src/process/posix_spawn.c
index ae262f7e..af127317 100644
--- a/src/process/posix_spawn.c
+++ b/src/process/posix_spawn.c
@@ -102,8 +102,7 @@ static int child(void *args_vp)
}
switch(op->cmd) {
case FDOP_CLOSE:
- if ((ret=__syscall(SYS_close, op->fd)))
- goto fail;
+ __syscall(SYS_close, op->fd);
break;
case FDOP_DUP2:
if ((ret=__sys_dup2(op->srcfd, op->fd))<0)