diff options
author | Joel Brobecker <brobecker@gnat.com> | 2008-05-20 18:36:36 +0000 |
---|---|---|
committer | Joel Brobecker <brobecker@gnat.com> | 2008-05-20 18:36:36 +0000 |
commit | c57918b25bb19d8ecaaad7eee1d372ffaf44d857 (patch) | |
tree | 0dcdd24a3bf90716347431d63f2d3f8ab6eb18b2 /gdb/win32-nat.c | |
parent | 9f1d377b33ab688f86e1cc9a454d87f991d65f19 (diff) | |
download | gdb-c57918b25bb19d8ecaaad7eee1d372ffaf44d857.zip gdb-c57918b25bb19d8ecaaad7eee1d372ffaf44d857.tar.gz gdb-c57918b25bb19d8ecaaad7eee1d372ffaf44d857.tar.bz2 |
* win32-nat.c (win32_wait): Block the control-c event while
waiting for a debug event.
Diffstat (limited to 'gdb/win32-nat.c')
-rw-r--r-- | gdb/win32-nat.c | 20 |
1 files changed, 19 insertions, 1 deletions
diff --git a/gdb/win32-nat.c b/gdb/win32-nat.c index d597b69..52ff20a 100644 --- a/gdb/win32-nat.c +++ b/gdb/win32-nat.c @@ -1458,7 +1458,25 @@ win32_wait (ptid_t ptid, struct target_waitstatus *ourstatus) while (1) { - int retval = get_win32_debug_event (pid, ourstatus); + int retval; + + /* Ignore CTRL+C signals while waiting for a debug event. + FIXME: brobecker/2008-05-20: When the user presses CTRL+C while + the inferior is running, both the inferior and GDB receive the + associated signal. If the inferior receives the signal first + and the delay until GDB receives that signal is sufficiently long, + GDB can sometimes receive the SIGINT after we have unblocked + the CTRL+C handler. This would lead to the debugger to stop + prematurely while handling the new-thread event that comes + with the handling of the SIGINT inside the inferior, and then + stop again immediately when the user tries to resume the execution + in the inferior. This is a classic race, and it would be nice + to find a better solution to that problem. But in the meantime, + the current approach already greatly mitigate this issue. */ + SetConsoleCtrlHandler (NULL, TRUE); + retval = get_win32_debug_event (pid, ourstatus); + SetConsoleCtrlHandler (NULL, FALSE); + if (retval) return pid_to_ptid (retval); else |