[Bug 53474] New: Change in memory handling gives crash in Framemaker 8
https://bugs.winehq.org/show_bug.cgi?id=53474 Bug ID: 53474 Summary: Change in memory handling gives crash in Framemaker 8 Product: Wine Version: 7.14 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: kernel32 Assignee: wine-bugs(a)winehq.org Reporter: ulrich.gemkow(a)ikr.uni-stuttgart.de Distribution: --- The following commit commit a4966b6690b31c4884d07a8a745017adb6363834 Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Jun 9 14:13:53 2022 +0200 kernelbase: Cleanup and simplify (Global|Local)ReAlloc. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> dlls/kernel32/heap.c | 22 ++++++++- dlls/kernel32/tests/heap.c | 15 ------- dlls/kernelbase/memory.c | 108 +++++++++++++++------------------------------ 3 files changed, 56 insertions(+), 89 deletions(-) leads to a crash in Framemaker 8. Reverting the patch on top of wine-7.14 solves the problem. To reproduce (simplest way): - Start Framemaker and create a new document - Open the print dialog (Ctrl-P) - Select another printer (lower part of the dialog) - Framemaker crashes with an internal error Framemaker generates a log file (which is IMHO useless but attached anyway) I can provide a demo version of framemaker if this would help. -- 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=53474 Ulrich Gemkow <ulrich.gemkow(a)ikr.uni-stuttgart.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression Regression SHA1| |a4966b6690b31c4884d07a8a745 | |017adb6363834 -- 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=53474 --- Comment #1 from Ulrich Gemkow <ulrich.gemkow(a)ikr.uni-stuttgart.de> --- Created attachment 72840 --> https://bugs.winehq.org/attachment.cgi?id=72840 Log File of Framemaker created after crash -- 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=53474 Ulrich Gemkow <ulrich.gemkow(a)ikr.uni-stuttgart.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rbernon(a)codeweavers.com -- 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=53474 --- Comment #2 from Rémi Bernon <rbernon(a)codeweavers.com> --- Would you mind testing again after fe3c5e93e21d72789a98a3d9ab8031146e56b058 (for instance today's upstream HEAD)? If it still doesn't work, a log file with WINEDEBUG=+pid,+globalmem could possibly be helpful. -- 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=53474 --- Comment #3 from Ulrich Gemkow <ulrich.gemkow(a)ikr.uni-stuttgart.de> --- Created attachment 73464 --> https://bugs.winehq.org/attachment.cgi?id=73464 Log File of Framemaker created after crash with +pid,+globalmem -- 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=53474 --- Comment #4 from Ulrich Gemkow <ulrich.gemkow(a)ikr.uni-stuttgart.de> --- Thanks for looking into this! Unfortunately the mentioned changes to the memory handling did not solve the problem. I attached the log file using latest wine 7.21 from yesterday. Please tell me when more info would help -- 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=53474 --- Comment #5 from Rémi Bernon <rbernon(a)codeweavers.com> --- Created attachment 73525 --> https://bugs.winehq.org/attachment.cgi?id=73525 GlobalReAlloc/LocalReAlloc fixes Would mind trying the attached patch, see if it fixes the issue? -- 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=53474 --- Comment #6 from Ulrich Gemkow <ulrich.gemkow(a)ikr.uni-stuttgart.de> --- Your patch indeed fixes the issue. Thanks again! -- 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=53474 Rémi Bernon <rbernon(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |03ad9474201b0ea20ec2282320c | |19bfe944788e8 Status|UNCONFIRMED |RESOLVED Resolution|--- |FIXED --- Comment #7 from Rémi Bernon <rbernon(a)codeweavers.com> --- This should be fixed after 03ad9474201b0ea20ec2282320c19bfe944788e8. -- 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=53474 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 7.22. -- 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=53474 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dark.shadow4(a)web.de --- Comment #9 from Fabian Maurer <dark.shadow4(a)web.de> --- Since part of those changes got reverted, does the program still work? -- 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=53474 --- Comment #10 from Ulrich Gemkow <ulrich.gemkow(a)ikr.uni-stuttgart.de> --- With current git of today (wine-7.22-223-g4febd9d880a) there are no crashes in this area. After the mentioned "fixed by" patch there were several changes in memory handling (if I understand this correctly) and more tests. Maybe in the end memory handling got improved. Please tell me if you need more info! -- 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=53474 --- Comment #11 from Fabian Maurer <dark.shadow4(a)web.de> --- No, if it works everything is great :) -- 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