diff options
| author | Rich Felker <dalias@aerifal.cx> | 2014-12-05 21:15:41 -0500 |
|---|---|---|
| committer | Rich Felker <dalias@aerifal.cx> | 2014-12-05 21:15:41 -0500 |
| commit | 1c12c24364d1058ffdbb28fca72a51de85082778 (patch) | |
| tree | c199e4d80a2f97a14e5f310979d8611a57227505 /src/process | |
| parent | 014275b547e3059db5c45986408757c250e8198d (diff) | |
| download | musl-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.c | 3 |
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) |