libs/newlib-cygwin
libs
/
newlib-cygwin
mirror of git://sourceware.org/git/newlib-cygwin.git
4
0
Fork 0
Code Issues Projects Releases Wiki Activity

Cygwin: console Add comment for the recent change 

Fixes: 30d2669478 ("Cygwin: console: Fix clean up conditions in close()")
Suggested-by: Brian Inglis <Brian.Inglis@SystematicSW.ab.ca>
Signed-off-by: Takashi Yano <takashi.yano@nifty.ne.jp>
This commit is contained in:
Takashi Yano 2024-11-11 21:38:38 +09:00
parent 6876520793
commit 87cd4f3fbd
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
winsup/cygwin/fhandler/console.cc
View File

@ -1984,6 +1984,15 @@ fhandler_console::close ()
NTSTATUS status;
status = NtQueryObject (get_handle (), ObjectBasicInformation,
&obi, sizeof obi, NULL);
/* If the process is not myself->cygstarted and is the console owner,
the process is the last process on this console device. The console
owner has two console handles, i.e. one is io_handle and the other
is the dupplicated handle for cons_master_thread.
If myself->cygstarted is false and the process is not console owner,
the process is supposed to be started by the exec command in the
owner shell. In this case, the owner process is still alive in the
background and waiting for this process. So the handle count is
three (two in the owner process, one is mine). */
if (NT_SUCCESS (status)
&& obi.HandleCount == (con.owner == GetCurrentProcessId () ? 2 : 3))
{