libs/newlib-cygwin
libs/newlib-cygwin
4
0
Fork 0
mirror of git://sourceware.org/git/newlib-cygwin.git synced 2025-02-08 18:19:08 +08:00
Code Issues Projects Releases Wiki Activity

Cygwin: console: Add missing guard regarding attach_mutex.

Browse Source 
- The commit a5333345 did not fix the problem enough. This patch
  provides additional guard for the issue.
This commit is contained in:
Takashi Yano via Cygwin-patches 2021-01-25 18:18:11 +09:00 committed by Ken Brown
parent 71c112d641
commit 0a7e412a19
1 changed files with 27 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
winsup/cygwin/fhandler_console.cc
View File

@ -439,14 +439,18 @@ fhandler_console::set_raw_win32_keyboard_mode (bool new_mode)
void
fhandler_console::set_cursor_maybe ()
{
acquire_attach_mutex (INFINITE);
con.fillin (get_output_handle ());
release_attach_mutex ();
/* Nothing to do for xterm compatible mode. */
if (wincap.has_con_24bit_colors () && !con_is_legacy)
return;
if (con.dwLastCursorPosition.X != con.b.dwCursorPosition.X ||
con.dwLastCursorPosition.Y != con.b.dwCursorPosition.Y)
{
acquire_attach_mutex (INFINITE);
SetConsoleCursorPosition (get_output_handle (), con.b.dwCursorPosition);
release_attach_mutex ();
con.dwLastCursorPosition = con.b.dwCursorPosition;
}
}
@ -536,6 +540,7 @@ fhandler_console::read (void *pv, size_t& buflen)
}
set_cursor_maybe (); /* to make cursor appear on the screen immediately */
wait_retry:
switch (cygwait (get_handle (), timeout))
{
case WAIT_OBJECT_0:
@ -551,6 +556,15 @@ fhandler_console::read (void *pv, size_t& buflen)
buflen = (size_t) -1;
return;
default:
if (GetLastError () == ERROR_INVALID_HANDLE)
{ /* Confirm the handle is still valid */
DWORD mode;
acquire_attach_mutex (INFINITE);
BOOL res = GetConsoleMode (get_handle (), &mode);
release_attach_mutex ();
if (res)
goto wait_retry;
}
goto err;
}
@ -1302,7 +1316,9 @@ fhandler_console::ioctl (unsigned int cmd, void *arg)
case TIOCGWINSZ:
int st;
acquire_attach_mutex (INFINITE);
st = con.fillin (get_output_handle ());
release_attach_mutex ();
if (st)
{
/* *not* the buffer size, the actual screen size... */
@ -1360,12 +1376,15 @@ fhandler_console::ioctl (unsigned int cmd, void *arg)
DWORD n;
int ret = 0;
INPUT_RECORD inp[INREC_SIZE];
acquire_attach_mutex (INFINITE);
if (!PeekConsoleInputW (get_handle (), inp, INREC_SIZE, &n))
{
set_errno (EINVAL);
release_attach_mutex ();
release_output_mutex ();
return -1;
}
release_attach_mutex ();
bool saw_eol = false;
for (DWORD i=0; i<n; i++)
if (inp[i].EventType == KEY_EVENT &&
@ -1416,11 +1435,13 @@ fhandler_console::tcflush (int queue)
if (queue == TCIFLUSH
|| queue == TCIOFLUSH)
{
acquire_attach_mutex (INFINITE);
if (!FlushConsoleInputBuffer (get_handle ()))
{
__seterrno ();
res = -1;
}
release_attach_mutex ();
}
return res;
}
@ -1434,12 +1455,14 @@ fhandler_console::output_tcsetattr (int, struct termios const *t)
DWORD flags = ENABLE_PROCESSED_OUTPUT | ENABLE_WRAP_AT_EOL_OUTPUT;
DWORD oflags;
acquire_attach_mutex (INFINITE);
GetConsoleMode (get_output_handle (), &oflags);
if (wincap.has_con_24bit_colors () && !con_is_legacy
&& (oflags & ENABLE_VIRTUAL_TERMINAL_PROCESSING))
flags |= ENABLE_VIRTUAL_TERMINAL_PROCESSING;
int res = SetConsoleMode (get_output_handle (), flags) ? 0 : -1;
release_attach_mutex ();
if (res)
__seterrno_from_win_error (GetLastError ());
release_output_mutex ();
@ -1457,6 +1480,7 @@ fhandler_console::input_tcsetattr (int, struct termios const *t)
DWORD oflags;
acquire_attach_mutex (INFINITE);
if (!GetConsoleMode (get_handle (), &oflags))
oflags = 0;
DWORD flags = 0;
@ -1515,6 +1539,7 @@ fhandler_console::input_tcsetattr (int, struct termios const *t)
syscall_printf ("%d = tcsetattr(,%p) enable flags %y, c_lflag %y iflag %y",
res, t, flags, t->c_lflag, t->c_iflag);
}
release_attach_mutex ();
get_ttyp ()->rstcons (false);
release_input_mutex ();
@ -1540,6 +1565,7 @@ fhandler_console::tcgetattr (struct termios *t)
DWORD flags;
acquire_attach_mutex (INFINITE);
if (!GetConsoleMode (get_handle (), &flags))
{
__seterrno ();
@ -1564,6 +1590,7 @@ fhandler_console::tcgetattr (struct termios *t)
/* All the output bits we can ignore */
res = 0;
}
release_attach_mutex ();
syscall_printf ("%d = tcgetattr(%p) enable flags %y, t->lflag %y, t->iflag %y",
res, t, flags, t->c_lflag, t->c_iflag);
return res;