https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=dfa09739d737ee26dbb40683dbf46865c99ff915
commit dfa09739d737ee26dbb40683dbf46865c99ff915
Author: Jeremy Drake <cyg...@jdrake.com>
Date: Mon Jul 7 11:26:37 2025 -0700
Cygwin: spawn: apply PID_NEW_PG flag to spawned child
Previously, it was always applied to "myself", which is only appropriate
in the case of _P_OVERLAY (aka exec). Apply the flag to "myself" only
in that case, and apply it to the new child instead in other cases.
Also, clear the flag from "myself" in the case of a failed exec.
Fixes: 8d8724ee1b5a ("Cygwin: pty: Fix Ctrl-C handling further for
non-cygwin apps.")
Signed-off-by: Jeremy Drake <cyg...@jdrake.com>
Diff:
---
winsup/cygwin/spawn.cc | 17 +++++++++++------
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/winsup/cygwin/spawn.cc b/winsup/cygwin/spawn.cc
index ef175e708..cb58b6eed 100644
--- a/winsup/cygwin/spawn.cc
+++ b/winsup/cygwin/spawn.cc
@@ -544,9 +544,6 @@ child_info_spawn::worker (const char *prog_arg, const char
*const *argv,
c_flags |= CREATE_NEW_PROCESS_GROUP;
refresh_cygheap ();
- if (c_flags & CREATE_NEW_PROCESS_GROUP)
- InterlockedOr ((LONG *) &myself->process_state, PID_NEW_PG);
-
if (mode == _P_DETACH)
/* all set */;
else if (mode != _P_OVERLAY || !my_wr_proc_pipe)
@@ -605,7 +602,12 @@ child_info_spawn::worker (const char *prog_arg, const char
*const *argv,
::cygheap->user.deimpersonate ();
if (!real_path.iscygexec () && mode == _P_OVERLAY)
- InterlockedOr ((LONG *) &myself->process_state, PID_NOTCYGWIN);
+ {
+ LONG pidflags = PID_NOTCYGWIN;
+ if (c_flags & CREATE_NEW_PROCESS_GROUP)
+ pidflags |= PID_NEW_PG;
+ InterlockedOr ((LONG *) &myself->process_state, pidflags);
+ }
cygpid = (mode != _P_OVERLAY) ? create_cygwin_pid () : myself->pid;
@@ -707,7 +709,8 @@ child_info_spawn::worker (const char *prog_arg, const char
*const *argv,
myself->sendsig = myself->exec_sendsig;
myself->exec_sendsig = NULL;
}
- InterlockedAnd ((LONG *) &myself->process_state, ~PID_NOTCYGWIN);
+ InterlockedAnd ((LONG *) &myself->process_state,
+ ~(PID_NOTCYGWIN|PID_NEW_PG));
/* Reset handle inheritance to default when the execution of a'
non-Cygwin process fails. Only need to do this for _P_OVERLAY
since the handle will be closed otherwise. Don't need to do
@@ -769,7 +772,9 @@ child_info_spawn::worker (const char *prog_arg, const char
*const *argv,
myself->set_has_pgid_children ();
ProtectHandle (pi.hThread);
pinfo child (cygpid,
- PID_IN_USE | (real_path.iscygexec () ? 0 :
PID_NOTCYGWIN));
+ PID_IN_USE |
+ (real_path.iscygexec () ? 0 : PID_NOTCYGWIN) |
+ ((c_flags & CREATE_NEW_PROCESS_GROUP) ? PID_NEW_PG : 0));
if (!child)
{
syscall_printf ("pinfo failed");
