spawn:posix: Take care of POSIX semaphore "shared" semantics.gniib/t8065

* src/spawn-posix.c (spawn_detached): Don't call _gpgrt_post_syscall
on the child process.
(_gpgrt_process_spawn): Likewise.

--

On NetBSD, a POSIX semaphore is somehow shared among processes, even
if sem_init is called with PSHARED=0.

GnuPG-bug-id: 8065
Signed-off-by: NIIBE Yutaka <[email protected]>

1 files changed, 4 insertions, 2 deletions

diff --git a/src/spawn-posix.c b/src/spawn-posix.c
index bd236ad..55f6379 100644
--- a/src/spawn-posix.c
+++ b/src/spawn-posix.c
@@ -460,9 +460,9 @@ spawn_detached (const char *pgmname, const char *argv[],
 
   _gpgrt_pre_syscall ();
   pid = fork ();
-  _gpgrt_post_syscall ();
   if (pid == (pid_t)(-1))
     {
+      _gpgrt_post_syscall ();
       ec = _gpg_err_code_from_syserror ();
       _gpgrt_log_info (_("error forking process: %s\n"), _gpg_strerror (ec));
       xfree (argv);
@@ -486,6 +486,7 @@ spawn_detached (const char *pgmname, const char *argv[],
       /*NOTREACHED*/
     }
 
+  _gpgrt_post_syscall ();
   xfree (argv);
   _gpgrt_pre_syscall ();
   if (waitpid (pid, NULL, 0) == -1)
@@ -752,9 +753,9 @@ _gpgrt_process_spawn (const char *pgmname, const char *argv1[],
 
   _gpgrt_pre_syscall ();
   pid = fork ();
-  _gpgrt_post_syscall ();
   if (pid == (pid_t)(-1))
     {
+      _gpgrt_post_syscall ();
       ec = _gpg_err_code_from_syserror ();
       _gpgrt_log_info (_("error forking process: %s\n"), _gpg_strerror (ec));
       if (fd_in[0] >= 0 && fd_in[0] != 0)
@@ -802,6 +803,7 @@ _gpgrt_process_spawn (const char *pgmname, const char *argv1[],
       /*NOTREACHED*/
     }
 
+  _gpgrt_post_syscall ();
   xfree (argv);
   process->pid = pid;