drew
/
acadia-newlib
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

* pinfo.cc (_pinfo::dup_proc_pipe): Make warning more severe by actually 

printing it.
(_pinfo::alert_parent): Fix comment.  Don't send to parent if we are in an exec
stub.
This commit is contained in:
Christopher Faylor 2005-10-19 14:19:37 +00:00
parent b72a5c0767
commit b3982520d3
2 changed files with 15 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
winsup/cygwin/ChangeLog
View File

@ -1,3 +1,10 @@
2005-10-19 Christopher Faylor <cgf@timesys.com>
* pinfo.cc (_pinfo::dup_proc_pipe): Make warning more severe by
actually printing it.
(_pinfo::alert_parent): Fix comment. Don't send to parent if we are in
an exec stub.
2005-10-18 Christopher Faylor <cgf@timesys.com> 2005-10-18 Christopher Faylor <cgf@timesys.com>
* cygtls.cc (handle_threadlist_exception): Improve diagnostic output. * cygtls.cc (handle_threadlist_exception): Improve diagnostic output.

13
winsup/cygwin/pinfo.cc
View File

@ -929,7 +929,7 @@ _pinfo::dup_proc_pipe (HANDLE hProcess)
bool res = DuplicateHandle (hMainProc, wr_proc_pipe, hProcess, &wr_proc_pipe, bool res = DuplicateHandle (hMainProc, wr_proc_pipe, hProcess, &wr_proc_pipe,
0, FALSE, flags); 0, FALSE, flags);
if (!res && WaitForSingleObject (hProcess, 0) != WAIT_OBJECT_0) if (!res && WaitForSingleObject (hProcess, 0) != WAIT_OBJECT_0)
sigproc_printf ("DuplicateHandle failed, pid %d, hProcess %p, %E", pid, hProcess); system_printf ("DuplicateHandle failed, pid %d, hProcess %p, %E", pid, hProcess);
else else
{ {
wr_proc_pipe_owner = dwProcessId; wr_proc_pipe_owner = dwProcessId;
@ -990,10 +990,13 @@ bool
_pinfo::alert_parent (char sig) _pinfo::alert_parent (char sig)
{ {
DWORD nb = 0; DWORD nb = 0;
/* Send something to our parent. If the parent has gone away,
close the pipe. */ /* Send something to our parent. If the parent has gone away, close the pipe.
if (wr_proc_pipe == INVALID_HANDLE_VALUE Don't send if this is an exec stub.
|| !myself->wr_proc_pipe)
FIXME: Is there a race here if we run this while another thread is attempting
to exec()? */
if (wr_proc_pipe == INVALID_HANDLE_VALUE || !myself->wr_proc_pipe || hExeced)
/* no parent */; /* no parent */;
else else
{ {