[Bug 50304] New: Control-C exits winedbg instead of stopping the inferior
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(a)winehq.org Reporter: joachim.priesner.bugs(a)web.de CC: jacek(a)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(a)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(a)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. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50304 --- Comment #1 from Joachim Priesner <joachim.priesner.bugs(a)web.de> --- Created attachment 68876 --> https://bugs.winehq.org/attachment.cgi?id=68876 +winedbg log of exit behavior -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50304 Gijs Vermeulen <gijsvrm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50304 --- Comment #2 from Jacek Caban <jacek(a)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. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50304 --- Comment #3 from Joachim Priesner <joachim.priesner.bugs(a)web.de> --- Thank you, the attached patch works for me as well. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50304 Jacek Caban <jacek(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |d7d315ddf40b22158fba76b1156 | |ffe09770103de Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED --- Comment #4 from Jacek Caban <jacek(a)codeweavers.com> --- Fixed in git, thanks for the report: https://source.winehq.org/git/wine.git/commitdiff/d7d315ddf40b22158fba76b115... -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50304 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #5 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 6.0-rc5. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=50304 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|unspecified |5.20 CC| |focht(a)gmx.net -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla