From 4ee52924a61bdb2fd8ce7b64d111cf7df4d19fe3 Mon Sep 17 00:00:00 2001 From: Christopher Faylor Date: Thu, 23 Dec 2004 14:57:08 +0000 Subject: * cygthread.cc (cygthread::stub): Detect if thread function wants to release itself here, to avoid a race. (cygthread::release): Clear more stuff. Add a diagnostic for an internal error. * cygthread.h (auto_release): New function. * pinfo.h (pinfo::remember): Add an argument to denote whether child is detached. * fork.cc (fork_parent): Reflect change in arguments to pinfo::remember. * pinfo.cc (_pinfo::exit): Signal exit more forcibly. (proc_waiter): Use cygthread::auto_release to signify that cygthread::stub should release the thread. This should avoid a race. (pinfo::alert_parent): Don't signify an error when wr_proc_pipe == NULL. * sigproc.cc (proc_subproc): Add support for PROC_DETACHED_CHILD. * sigproc.h: Ditto. * spawn.cc (spawn_guts): Specify whether child is detached or not when calling pinfo::remember. --- winsup/cygwin/spawn.cc | 22 ++++++++++------------ 1 file changed, 10 insertions(+), 12 deletions(-) (limited to 'winsup/cygwin/spawn.cc') diff --git a/winsup/cygwin/spawn.cc b/winsup/cygwin/spawn.cc index 587c963cc..1c84fad4e 100644 --- a/winsup/cygwin/spawn.cc +++ b/winsup/cygwin/spawn.cc @@ -802,7 +802,7 @@ spawn_guts (const char * prog_arg, const char *const *argv, this). */ if (!myself->wr_proc_pipe) { - myself.remember (); + myself.remember (true); wait_for_myself = true; myself->wr_proc_pipe = INVALID_HANDLE_VALUE; } @@ -822,14 +822,6 @@ spawn_guts (const char * prog_arg, const char *const *argv, } child->dwProcessId = pi.dwProcessId; child.hProcess = pi.hProcess; - if (!child.remember ()) - { - /* FIXME: Child in strange state now. */ - CloseHandle (pi.hProcess); - CloseHandle (pi.hThread); - res = -1; - goto out; - } strcpy (child->progname, real_path); /* FIXME: This introduces an unreferenced, open handle into the child. @@ -839,9 +831,15 @@ spawn_guts (const char * prog_arg, const char *const *argv, However, we should try to find another way to do this eventually. */ (void) DuplicateHandle (hMainProc, child.shared_handle (), pi.hProcess, NULL, 0, 0, DUPLICATE_SAME_ACCESS); - if (mode == _P_DETACH) - myself.alert_parent (0); child->start_time = time (NULL); /* Register child's starting time. */ + if (!child.remember (mode == _P_DETACH)) + { + /* FIXME: Child in strange state now */ + CloseHandle (pi.hProcess); + ForceCloseHandle (pi.hThread); + res = -1; + goto out; + } } /* Start the child running */ @@ -867,7 +865,7 @@ switch (mode) res = -1; break; case _P_DETACH: - res = 0; /* Lose all memory of this child. */ + res = 0; /* Lost all memory of this child. */ break; case _P_NOWAIT: case _P_NOWAITO: -- cgit v1.2.3