http://bugs.winehq.org/show_bug.cgi?id=21069
Summary: 16 bit apps crashes on exit Product: Wine Version: 1.1.35 Platform: x86 URL: http://www.arcelectronicsinc.com/testinfo/kpnl1tester. zip OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: wylda@volny.cz
Created an attachment (id=25267) --> (http://bugs.winehq.org/attachment.cgi?id=25267) console log
During testing & helping with bug 21063 i discovered another regression.
1. Easily reproducible - when you run KPNL1.EXE it will crash on exit.
2. Did a regression test between 1.1.34 and 1.1.35:
fc0244cb32b5db1b075fd008a23e3c35324b9a96 is first bad commit commit fc0244cb32b5db1b075fd008a23e3c35324b9a96 Author: Alexandre Julliard julliard@winehq.org Date: Tue Dec 15 20:11:16 2009 +0100
user32: Store the 16-bit winproc in a different array from the 32-bit A/W ones.
:040000 040000 1425d416c9ac20628c63e33a3a37e3d746fa9ffc a5454e27aa45eea0ad05a7502218ec13e1378baa M dlls
3. No other bug report suffers from this commit.
4. Revert of this patch after git checkout makes that problem goes away.
5. Adding author of this patch to CC.
--private keyword: bisected
http://bugs.winehq.org/show_bug.cgi?id=21069
Wylda wylda@volny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, regression, win16 CC| |julliard@winehq.org
http://bugs.winehq.org/show_bug.cgi?id=21069
Paul Romanyszyn pgr@arcelectronicsinc.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |pgr@arcelectronicsinc.com
--- Comment #1 from Paul Romanyszyn pgr@arcelectronicsinc.com 2009-12-18 18:08:08 --- Thanks for doing the testing. There is an non regression crash on exit that as I recall from when I started using wine with this application around 2005. Under windows you can start tester.exe and it will load the dll and kpnl1.exe. Closing tester.exe should leave the dll and kpnl1.exe running and restarting tester.exe should locate kpnl1.exe and connect to it. I never filed a bug for that as an exit crash was not as important as start up and running crashes.
http://bugs.winehq.org/show_bug.cgi?id=21069
--- Comment #2 from Wylda wylda@volny.cz 2009-12-18 18:36:20 ---
(In reply to comment #1)
Thanks for doing the testing. There is an non regression crash on exit...
Well this is a regression. I did not test "tester.exe", but only "KPNL1.EXE". Under wine 1.1.34 i can ran it and cleanly exit. These two things are not possible under 1.1.35. If there is some other problem, it should NOT be mentioned here, but in a new bug report (1problem=1bug report).
Maybe it would be good to wait, than this bug and bug 21063 will be fixed and then try another things with tester.exe (it's not usable now).
http://bugs.winehq.org/show_bug.cgi?id=21069
--- Comment #3 from Wylda wylda@volny.cz 2009-12-19 18:05:34 --- Created an attachment (id=25289) --> (http://bugs.winehq.org/attachment.cgi?id=25289) Civilization II's console log from wine-1.1.35
Commit in Comment #0 also affect installation of Sid Mayer's Civilization 2. After selecting English, it should open "Installation Read me", but instead of it, wine-1.1.35 crashes.
Different console output on same commit, so attaching.
http://bugs.winehq.org/show_bug.cgi?id=21069
--- Comment #4 from Wylda wylda@volny.cz 2009-12-22 05:15:55 ---
Still not fixed in wine-1.1.35-56-g13a9c03.
http://bugs.winehq.org/show_bug.cgi?id=21069
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED
--- Comment #5 from Alexandre Julliard julliard@winehq.org 2009-12-22 09:59:22 --- Should be fixed now.
http://bugs.winehq.org/show_bug.cgi?id=21069
André H. nerv@dawncrow.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |aferre@gmail.com
--- Comment #6 from André H. nerv@dawncrow.de 2009-12-22 10:25:44 --- *** Bug 21100 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=21069
--- Comment #7 from Wylda wylda@volny.cz 2009-12-22 12:43:28 ---
Confirming - FIXED. So Paul, you spoke (comment #1) about some other crash. Could you retest your app with latest git version and if it still crashes, write simple "how to" reproduce that crash in a newly opened bug report?
http://bugs.winehq.org/show_bug.cgi?id=21069
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |saulius2@gmail.com
--- Comment #8 from Alexandre Julliard julliard@winehq.org 2009-12-22 16:32:22 --- *** Bug 21121 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=21069
--- Comment #9 from Saulius K. saulius2@gmail.com 2009-12-23 14:45:04 --- Cool! It wasn't my bug 21121 report finished, yet the bug was fixed by Alexandre [*]:
Author: Alexandre Julliard julliard@winehq.org Date: Mon Dec 21 19:48:12 2009 +0100
user32: Always return the actual pointer instead of a winproc handle in WINPROC_GetProc16.
Thank you.
[*] http://source.winehq.org/git/wine.git/?a=commit;h=5a35c490a186d3760840a72af8
http://bugs.winehq.org/show_bug.cgi?id=21069
Jeff Zaroyko jeffz@jeffz.name changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #10 from Jeff Zaroyko jeffz@jeffz.name 2010-01-09 04:50:42 --- Closing bugs fixed in 1.1.36.
http://bugs.winehq.org/show_bug.cgi?id=21069
Wylda wylda@volny.cz changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |5a35c490a186d3760840a72af89 | |cd6aafd926f05 Regression SHA1| |fc0244cb32b5db1b075fd008a23 | |e3c35324b9a96
-
wine-bugs＠winehq.org