https://bugs.winehq.org/show_bug.cgi?id=53999
Bug ID: 53999 Summary: Regression with JGlossator Product: Wine Version: 7.22 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: mshtml Assignee: wine-bugs@winehq.org Reporter: jkfloris@dds.nl Distribution: ---
https://forum.winehq.org/viewtopic.php?t=37152
Although I don't know exactly what the program is supposed to do, it no longer works with Wine 7.22
JGlossator v5 https://sourceforge.net/projects/jglossator/files/JGlossator_v5.0/
bisect: 6fa4c426086750d616d5da9c41b37fc714784d1a is the first bad commit commit 6fa4c426086750d616d5da9c41b37fc714784d1a Author: Gabriel Ivăncescu gabrielopcode@gmail.com Date: Tue Nov 22 18:26:22 2022 +0200
mshtml: Get rid of useless window check.
Signed-off-by: Gabriel Ivăncescu gabrielopcode@gmail.com
dlls/mshtml/nsevents.c | 12 ++++--------
https://bugs.winehq.org/show_bug.cgi?id=53999
--- Comment #1 from jkfloris@dds.nl --- Created attachment 73583 --> https://bugs.winehq.org/attachment.cgi?id=73583 +mshtml good terminal output
https://bugs.winehq.org/show_bug.cgi?id=53999
--- Comment #2 from jkfloris@dds.nl --- Created attachment 73584 --> https://bugs.winehq.org/attachment.cgi?id=73584 +mshtml bad terminal output
https://bugs.winehq.org/show_bug.cgi?id=53999
jkfloris@dds.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |gabrielopcode@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=53999
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Regression with JGlossator |JGlossator crashes Keywords| |download, regression Regression SHA1| |6fa4c426086750d616d5da9c41b | |37fc714784d1a
https://bugs.winehq.org/show_bug.cgi?id=53999
Gabriel Ivăncescu gabrielopcode@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Ever confirmed|0 |1 Status|UNCONFIRMED |ASSIGNED
--- Comment #3 from Gabriel Ivăncescu gabrielopcode@gmail.com --- Sorry about this. The app seems to navigate during the notification callbacks in handle_load, so it detaches the document from the window (since we don't keep a ref to prevent circular refcounts), even if we keep a ref to the document itself.
Proper fix would be to keep a ref to the window also. I've added it to my latest MR.
https://bugs.winehq.org/show_bug.cgi?id=53999
Gabriel Ivăncescu gabrielopcode@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Assignee|wine-bugs@winehq.org |gabrielopcode@gmail.com
https://bugs.winehq.org/show_bug.cgi?id=53999
--- Comment #4 from jkfloris@dds.nl --- Confirming, MR1598 fixes this bug.
https://bugs.winehq.org/show_bug.cgi?id=53999
Gabriel Ivăncescu gabrielopcode@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|ASSIGNED |RESOLVED Fixed by SHA1| |ce54f50feb57dd3360e99a457db | |cf8d3dc0b0993
--- Comment #5 from Gabriel Ivăncescu gabrielopcode@gmail.com --- Fixed by ce54f50feb57dd3360e99a457dbcf8d3dc0b0993.
https://bugs.winehq.org/show_bug.cgi?id=53999
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #6 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 8.0-rc1.
-
WineHQ Bugzilla