summaryrefslogtreecommitdiff
path: root/src/libstddjb/cspawn_workaround.c
diff options
context:
space:
mode:
authorLaurent Bercot <ska-skaware@skarnet.org>2023-09-09 03:55:38 +0000
committerLaurent Bercot <ska@appnovation.com>2023-09-09 03:55:38 +0000
commit4cce87557feb6933c284d234c448bd8bde4facae (patch)
tree7417e8558b0770da2ae24df8abd755dbc913855f /src/libstddjb/cspawn_workaround.c
parenta646c96d5f7ec4b4e0bb538a41fc202c2cdb3309 (diff)
downloadskalibs-4cce87557feb6933c284d234c448bd8bde4facae.tar.xz
cspawn revamp, part 1. Prepare for 2.14.0.0.
Signed-off-by: Laurent Bercot <ska@appnovation.com>
Diffstat (limited to 'src/libstddjb/cspawn_workaround.c')
-rw-r--r--src/libstddjb/cspawn_workaround.c55
1 files changed, 55 insertions, 0 deletions
diff --git a/src/libstddjb/cspawn_workaround.c b/src/libstddjb/cspawn_workaround.c
new file mode 100644
index 0000000..da9bd89
--- /dev/null
+++ b/src/libstddjb/cspawn_workaround.c
@@ -0,0 +1,55 @@
+/* ISC license. */
+
+#include <skalibs/sysdeps.h>
+#include <skalibs/djbunix.h>
+
+#if defined(SKALIBS_HASPOSIXSPAWN) && defined(SKALIBS_HASPOSIXSPAWNEARLYRETURN)
+
+#include <errno.h>
+#include <signal.h>
+#include <sys/wait.h>
+
+#include <skalibs/allreadwrite.h>
+
+ /* when posix_spawn returns too early, you need this */
+
+pid_t cspawn_workaround (pid_t pid, int const *p)
+{
+ siginfo_t si ;
+ int e ;
+ ssize_t r ;
+ char c ;
+
+ fd_close(p[1]) ;
+ r = fd_read(p[0], &c, 1) ;
+ fd_close(p[0]) ;
+ if (r == -1) return 0 ;
+ if (r) return (errno = EILSEQ, 0) ; /* child wrote, wtf */
+
+ do e = waitid(P_PID, pid, &si, WEXITED | WNOHANG | WNOWAIT) ;
+ while (e == -1 && errno == EINTR) ;
+ if (e == -1) return pid ; /* we're in trouble, but don't leak a child */
+ if (!si.si_pid) return pid ; /* child is running */
+ if (si.si_code != CLD_EXITED || si.si_status != 127) return pid ; /* child died after execve(), let caller handle it */
+ /*
+ child exited 127, so either execve() failed, which is what we want to catch,
+ or it raced like a mofo, execve()d and then exited 127 on its own, in which
+ case, tough luck, it never existed.
+ */
+ wait_pid(pid, 0) ;
+ return (errno = 0, 0) ;
+}
+
+#else
+
+pid_t cspawn_workaround (pid_t pid, int const *p)
+{
+ if (p)
+ {
+ fd_close(p[1]) ;
+ fd_close(p[0]) ;
+ }
+ return pid ;
+}
+
+#endif