diff options
| author | Laurent Bercot <ska-skaware@skarnet.org> | 2019-06-16 09:01:54 +0000 |
|---|---|---|
| committer | Laurent Bercot <ska-skaware@skarnet.org> | 2019-06-16 09:01:54 +0000 |
| commit | da84f235ef985883c79066204ee838717aacd41a (patch) | |
| tree | b7a8dd8931814b9ad9a02bf848020645c0146cc1 /src/libstddjb/child_spawn1_internal.c | |
| parent | dae816d196ca006cb0e820de1d468e551129f220 (diff) | |
| download | skalibs-da84f235ef985883c79066204ee838717aacd41a.tar.xz | |
child_spawn*: make sure posix_spawn() doesn't clobber pipes with dup2+close
Diffstat (limited to 'src/libstddjb/child_spawn1_internal.c')
| -rw-r--r-- | src/libstddjb/child_spawn1_internal.c | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/libstddjb/child_spawn1_internal.c b/src/libstddjb/child_spawn1_internal.c index 0f9cb16..116a65e 100644 --- a/src/libstddjb/child_spawn1_internal.c +++ b/src/libstddjb/child_spawn1_internal.c @@ -32,10 +32,13 @@ pid_t child_spawn1_internal (char const *prog, char const *const *argv, char con } e = posix_spawn_file_actions_init(&actions) ; if (e) goto errattr ; - e = posix_spawn_file_actions_adddup2(&actions, p[to & 1], to & 1) ; - if (e) goto erractions ; - e = posix_spawn_file_actions_addclose(&actions, p[to & 1]) ; - if (e) goto erractions ; + if (p[to & 1] != to & 1) + { + e = posix_spawn_file_actions_adddup2(&actions, p[to & 1], to & 1) ; + if (e) goto erractions ; + e = posix_spawn_file_actions_addclose(&actions, p[to & 1]) ; + if (e) goto erractions ; + } if (to & 2) { e = posix_spawn_file_actions_adddup2(&actions, to & 1, !(to & 1)) ; |