[Bug 56032] New: winedbg --gdb: gets terminated when target process exits
https://bugs.winehq.org/show_bug.cgi?id=56032 Bug ID: 56032 Summary: winedbg --gdb: gets terminated when target process exits Product: Wine Version: 6.19 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: winedbg Assignee: wine-bugs(a)winehq.org Reporter: bernhardu(a)mailbox.org Distribution: --- Since below commit following sequence shows a termination of winedbg and leaves the terminal in a non-working state. Before after target process finished one was still at the winedbg prompt. benutzer(a)debian:~$ wine winedbg.exe --gdb notepad.exe ... Wine-gdb> cont ... [Inferior 1 (Remote target) exited normally] Wine-gdb> benutzer(a)debian:~$ benutzer(a)debian:~$ quit benutzer(a)debian:~$ commit 6a2325609ceea306c6d7cbce56f29e6589a18903 Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Oct 4 11:27:43 2021 +0200 winedbg: Use winsock for the gdb socket. The terminal can be recovered afterwards by "stty sane". -- 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=56032 Bernhard Übelacker <bernhardu(a)mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |6a2325609ceea306c6d7cbce56f | |29e6589a18903 Distribution|--- |Debian -- 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=56032 Bernhard Übelacker <bernhardu(a)mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression, source -- 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=56032 Eric Pouech <eric.pouech(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |eric.pouech(a)gmail.com --- Comment #1 from Eric Pouech <eric.pouech(a)gmail.com> --- what likely happens is that: - wine starts winedbg - wine hence attaches conhost to shell unix console; this saves the tty settings (setA), but changes also the tty settings (setB) for conhost to work properly - winedbg starts gdb - gdb starts (it saves the tty settings setC = setB, and puts its own settings in place (setD) - debuggee executes and terminates - for some reasons winedbg terminates first, which closes conhost, and conhost resets tty settings to setA - then gdb terminates, and it resets terminal settings to what it got upon startup = setB == setC -- 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=56032 --- Comment #2 from Eric Pouech <eric.pouech(a)gmail.com> --- Created attachment 75681 --> https://bugs.winehq.org/attachment.cgi?id=75681 tentative patch the attached patch solves the issue locally does it on your end too? -- 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=56032 Eric Pouech <eric.pouech(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |ASSIGNED Ever confirmed|0 |1 --- Comment #3 from Eric Pouech <eric.pouech(a)gmail.com> --- confirming -- 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=56032 --- Comment #4 from Bernhard Übelacker <bernhardu(a)mailbox.org> --- Thanks for looking into this. Your patch works perfectly and makes winedbg behave like before wine-6.19. -- 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=56032 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dark.shadow4(a)web.de --- Comment #5 from Fabian Maurer <dark.shadow4(a)web.de> --- Is this fixed by https://gitlab.winehq.org/wine/wine/-/commit/b12c1eaea6f1bd98a03eeed9b84701d... ? -- 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=56032 --- Comment #6 from Eric Pouech <eric.pouech(a)gmail.com> --- (In reply to Fabian Maurer from comment #5)
Is this fixed by https://gitlab.winehq.org/wine/wine/-/commit/ b12c1eaea6f1bd98a03eeed9b84701df5eb7b20c ?
In theory, it should (it's a slightly updated version of patch in #2). In practice, as Bernhard opened the ticket, I rather let him have the final approval on the fix. -- 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=56032 Bernhard Übelacker <bernhardu(a)mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|ASSIGNED |RESOLVED Fixed by SHA1| |b12c1eaea6f1bd98a03eeed9b84 | |701df5eb7b20c --- Comment #7 from Bernhard Übelacker <bernhardu(a)mailbox.org> --- Thanks, marking fixed. -- 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=56032 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #8 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 9.0-rc4. -- 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