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: pty: Close pseudo console only if the process is the owner. 

- Currently, close_pseudoconsole() is called unconditionally from
  fhandler_termios::process_sigs() on Ctrl-C. This causes deadlock
  if Ctrl-C is pressed while setup_pseudoconsole() is called. With
  this patch, close_pseudoconsole() is called only if the master
  process is the owner of the nat-pipe to avoid the deadlock.
This commit is contained in:
Takashi Yano 2022-04-09 12:59:54 +09:00
parent db1fbbc495
commit 45cfcfbd71
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
winsup/cygwin/fhandler_tty.cc
View File

@ -4152,7 +4152,8 @@ void
fhandler_pty_slave::release_ownership_of_nat_pipe (tty *ttyp, fhandler_pty_slave::release_ownership_of_nat_pipe (tty *ttyp,
fhandler_termios *fh) fhandler_termios *fh)
{ {
if (fh->get_major () == DEV_PTYM_MAJOR) if (fh->get_major () == DEV_PTYM_MAJOR
&& nat_pipe_owner_self (ttyp->nat_pipe_owner_pid))
{ {
fhandler_pty_master *ptym = (fhandler_pty_master *) fh; fhandler_pty_master *ptym = (fhandler_pty_master *) fh;
WaitForSingleObject (ptym->pipe_sw_mutex, INFINITE); WaitForSingleObject (ptym->pipe_sw_mutex, INFINITE);