https://bugs.winehq.org/show_bug.cgi?id=50304
Bug ID: 50304 Summary: Control-C exits winedbg instead of stopping the inferior Product: Wine Version: unspecified Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: winedbg Assignee: wine-bugs@winehq.org Reporter: joachim.priesner.bugs@web.de CC: jacek@codeweavers.com Regression SHA1: 633b244b1ae7928cafbfd25ef338e4918aac0d74 Distribution: ---
Created attachment 68875 --> https://bugs.winehq.org/attachment.cgi?id=68875 +winedbg log of hang behavior
To reproduce:
- run wine winedbg program.exe - press "c <Enter>" to run the program - press Control-C
Expected: The program is stopped and I am back at the winedbg prompt.
Actual:
The behavior matched the expected one until the following commit:
commit 54e117018cd4cc58c258da92686bfad13946bde2 Author: Jacek Caban jacek@codeweavers.com Date: Mon Sep 21 17:07:29 2020 +0200
kernelbase: Use conhost to handle Unix consoles.
Starting at this commit, pressing Control-C would cause winedbg to hang (needed to "kill -9" it).
The behavior changed again in the following commit:
commit 633b244b1ae7928cafbfd25ef338e4918aac0d74 Author: Jacek Caban jacek@codeweavers.com Date: Tue Oct 13 16:27:38 2020 +0200
kernel32: Always use conhost for ReadConsoleW.
Starting at this commit, and until the current master, pressing Control-C causes winedbg to immediately exit.
https://bugs.winehq.org/show_bug.cgi?id=50304
--- Comment #1 from Joachim Priesner joachim.priesner.bugs@web.de --- Created attachment 68876 --> https://bugs.winehq.org/attachment.cgi?id=68876 +winedbg log of exit behavior
https://bugs.winehq.org/show_bug.cgi?id=50304
Gijs Vermeulen gijsvrm@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression
https://bugs.winehq.org/show_bug.cgi?id=50304
--- Comment #2 from Jacek Caban jacek@codeweavers.com --- Created attachment 69036 --> https://bugs.winehq.org/attachment.cgi?id=69036 patch
conhost itself is not attached to any console in Windows sense, so it does not get the treatment by kernelbase and is terminated, causing termination of its clients, even if they handled the exception themselves. The attached patch fixes it for me. We can just ignore the signal in conhost.
https://bugs.winehq.org/show_bug.cgi?id=50304
--- Comment #3 from Joachim Priesner joachim.priesner.bugs@web.de --- Thank you, the attached patch works for me as well.
https://bugs.winehq.org/show_bug.cgi?id=50304
Jacek Caban jacek@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |d7d315ddf40b22158fba76b1156 | |ffe09770103de Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED
--- Comment #4 from Jacek Caban jacek@codeweavers.com --- Fixed in git, thanks for the report: https://source.winehq.org/git/wine.git/commitdiff/d7d315ddf40b22158fba76b115...
https://bugs.winehq.org/show_bug.cgi?id=50304
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #5 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 6.0-rc5.
https://bugs.winehq.org/show_bug.cgi?id=50304
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |5.20 CC| |focht@gmx.net