http://bugs.winehq.org/show_bug.cgi?id=22796
Summary: PopCap games installers crash in Win Vista+ mode. Product: Wine Version: 1.1.44 Platform: x86 URL: http://downloads.popcap.com/www/popcap_downloads/Astro PopSetup.exe OS/Version: Linux Status: NEW Keywords: download, Installer Severity: minor Priority: P3 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: kennybobs@o2.co.uk
Created an attachment (id=28131) --> (http://bugs.winehq.org/attachment.cgi?id=28131) wine-1.1.44-380-g0d84ccb console output
Tested AstroPop, Zuma and Zuma's Revenge. PopCap use the same installer for most their games so they're all likely to be affected.
In Windows Vista, 2008 and 7 mode, the installers immediately cause a page fault.
In Windows XP mode they install fine.
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #1 from Ken Sharp kennybobs@o2.co.uk 2010-05-20 13:16:46 --- Created an attachment (id=28132) --> (http://bugs.winehq.org/attachment.cgi?id=28132) wine-1.1.44-380-g0d84ccb +relay +seh +tid (217 kB)
http://bugs.winehq.org/show_bug.cgi?id=22796
Hans Leidekker hans@meelstraat.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |hans@meelstraat.net
--- Comment #2 from Hans Leidekker hans@meelstraat.net 2010-05-20 14:03:28 --- 0029:Call KERNEL32.LoadLibraryA(00446698 "shell32.dll") 0029:Ret KERNEL32.LoadLibraryA() retval=7e8c0000 0029:Call KERNEL32.GetProcAddress(7e8c0000,00448d28 "SHGetKnownFolderIDList") 0029:Ret KERNEL32.GetProcAddress() retval=00000000 0029:Call KERNEL32.GetProcAddress(7e8c0000,00448d40 "ILFree") 0029:Ret KERNEL32.GetProcAddress() retval=00000000 0029:Call KERNEL32.GetProcAddress(7e8c0000,00448d48 "SHGetLocalizedName") 0029:Ret KERNEL32.GetProcAddress() retval=00000000
SHGetKnownFolderIDList and SHGetLocalizedName were introduced in Vista and they are unimplemented in Wine. ILFree is implemented but exported by ordinal only.
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #3 from Nikolay Sivov bunglehead@gmail.com 2010-05-20 14:12:08 --- I'll prepare some stubs for that (after build completion).
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #4 from Nikolay Sivov bunglehead@gmail.com 2010-05-20 14:34:08 --- Created an attachment (id=28136) --> (http://bugs.winehq.org/attachment.cgi?id=28136) SHGetLocalizedName stub
Ken, please try this one first.
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #5 from Ken Sharp kennybobs@o2.co.uk 2010-05-20 15:08:52 --- Same crash with patch applied.
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #6 from Nikolay Sivov bunglehead@gmail.com 2010-05-20 15:17:09 --- (In reply to comment #5)
Same crash with patch applied.
OK, attach a new +relay,+seh,+tid with this patch and another one I just sent:
http://www.winehq.org/pipermail/wine-patches/2010-May/088727.html
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #7 from Ken Sharp kennybobs@o2.co.uk 2010-05-20 15:47:37 --- Created an attachment (id=28137) --> (http://bugs.winehq.org/attachment.cgi?id=28137) +relay +seh +tid with both patches
Same crash.
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #8 from Nikolay Sivov bunglehead@gmail.com 2010-05-22 05:10:12 --- Created an attachment (id=28172) --> (http://bugs.winehq.org/attachment.cgi?id=28172) Export some calls by name too
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #9 from Nikolay Sivov bunglehead@gmail.com 2010-05-22 05:11:20 --- Created an attachment (id=28173) --> (http://bugs.winehq.org/attachment.cgi?id=28173) SHGetKnownFolderIDList stub
Try two last patches and if it doesn't work attach an updated log.
http://bugs.winehq.org/show_bug.cgi?id=22796
Nikolay Sivov bunglehead@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #28136|0 |1 is obsolete| |
http://bugs.winehq.org/show_bug.cgi?id=22796
Vincent Povirk madewokherd@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |madewokherd@gmail.com
http://bugs.winehq.org/show_bug.cgi?id=22796
Nikolay Sivov bunglehead@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #28172|0 |1 is obsolete| |
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #10 from Nikolay Sivov bunglehead@gmail.com 2010-05-26 16:02:22 --- So another test needed with current wine-1.2-rc1-108-g5d4a743 and SHGetKnownFolderIDList patch (wasn't accepted yet).
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #11 from Ken Sharp kennybobs@o2.co.uk 2010-05-27 20:11:52 --- Hi Nikolay,
The SHGetKnownFolderIDList patch won't compile with wine-1.2-rc1-142-ga72f3f3, I don't know if it's something I've done wrong...
../../../dlls/shell32/shell32_main.c:1235: error: redefinition of ‘SHGetLocalizedName’ ../../../dlls/shell32/shell32_main.c:1229: note: previous definition of ‘SHGetLocalizedName’ was here make[1]: *** [shell32_main.o] Error 1 make[1]: Leaving directory `/home/test/wine-git/wine32/dlls/shell32' make: *** [dlls/shell32] Error 2
http://bugs.winehq.org/show_bug.cgi?id=22796
Nikolay Sivov bunglehead@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #28173|0 |1 is obsolete| |
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #12 from Nikolay Sivov bunglehead@gmail.com 2010-05-28 06:46:17 --- Created an attachment (id=28346) --> (http://bugs.winehq.org/attachment.cgi?id=28346) Updated SHGetKnownFolderIDList stub
Try this one with current wine.
http://bugs.winehq.org/show_bug.cgi?id=22796
Ken Sharp kennybobs@o2.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #28137|0 |1 is obsolete| |
--- Comment #13 from Ken Sharp kennybobs@o2.co.uk 2010-05-28 08:24:27 --- Created an attachment (id=28350) --> (http://bugs.winehq.org/attachment.cgi?id=28350) Screenshot of error with patch applied
The patch does indeed stop the crash, but leads to another minor bug.
The installer usually suggests where to install the application, but in this instance does not and produces an error.
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #14 from Ken Sharp kennybobs@o2.co.uk 2010-05-28 08:25:44 --- Created an attachment (id=28352) --> (http://bugs.winehq.org/attachment.cgi?id=28352) Console output with patch applied
Looks like a stub isn't enough to stop errors.
http://bugs.winehq.org/show_bug.cgi?id=22796
Nikolay Sivov bunglehead@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|-unknown |shell32
--- Comment #15 from Nikolay Sivov bunglehead@gmail.com 2010-05-28 10:13:32 --- Ok, thanks.
It wants a Users directory (if I remember its name on Vista correctly) path to use it as initial choice I believe. I'm setting this to shell32, we need to revisit this after 1.2 released (stub patch was deferred).
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #16 from Ken Sharp kennybobs@o2.co.uk 2011-04-20 17:03:35 CDT --- Still present in Wine 1.3.18.
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #17 from Ken Sharp kennybobs@o2.co.uk 2011-10-02 15:58:18 CDT --- Still present in Wine 1.3.29
http://bugs.winehq.org/show_bug.cgi?id=22796
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |focht@gmx.net Summary|PopCap games installers |PopCap games installers |crash in Win Vista+ mode. |crash with WinVer >= Vista | |(shell32 | |SHGetKnownFolderIDList stub | |needed)
--- Comment #18 from Anastasius Focht focht@gmx.net 2012-04-30 16:08:45 CDT --- Hello,
still present.
--- snip --- 0024:Call KERNEL32.LoadLibraryA(00446698 "shell32.dll") ret=0041711f 0024:Ret KERNEL32.LoadLibraryA() retval=7e7c0000 ret=0041711f 0024:Call KERNEL32.GetProcAddress(7e7c0000,00448d28 "SHGetKnownFolderIDList") ret=00417134 0024:Ret KERNEL32.GetProcAddress() retval=00000000 ret=00417134 0024:Call KERNEL32.GetProcAddress(7e7c0000,00448d40 "ILFree") ret=00417140 0024:Ret KERNEL32.GetProcAddress() retval=7e7c7eb4 ret=00417140 0024:Call KERNEL32.GetProcAddress(7e7c0000,00448d48 "SHGetLocalizedName") ret=0041714c 0024:Ret KERNEL32.GetProcAddress() retval=7e7c89f4 ret=0041714c 0024:trace:seh:raise_exception code=c0000005 flags=0 addr=0x7bc78e63 ip=7bc78e63 tid=0024 0024:trace:seh:raise_exception info[0]=00000000 0024:trace:seh:raise_exception info[1]=00000000 0024:trace:seh:raise_exception eax=00000000 ebx=7bcbf544 ecx=0396c227 edx=00000000 esi=0032f49c edi=0032f52c 0024:trace:seh:raise_exception ebp=0032ea40 esp=0032e948 cs=0023 ds=002b es=002b fs=0063 gs=006b flags=00010246 0024:trace:seh:call_stack_handlers calling handler at 0x7bc91c09 code=c0000005 flags=0 --- snip ---
A stub for SHGetKnownFolderIDList() would be a start.
$ du -sh AstroPopSetup.exe 8.0M AstroPopSetup.exe
$ sha1sum AstroPopSetup.exe 177d186b460ab4fa0f070d06f7ae25c69efa7791 AstroPopSetup.exe
$ wine --version wine-1.5.3-39-gd2581e6
Regards
http://bugs.winehq.org/show_bug.cgi?id=22796
--- Comment #19 from Austin English austinenglish@gmail.com 2012-04-30 18:08:36 CDT --- http://source.winehq.org/patches/data/85885
http://bugs.winehq.org/show_bug.cgi?id=22796
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |adc0518af74d79e45b68a4119d3 | |532c6c4e8599e Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #20 from Austin English austinenglish@gmail.com 2012-05-01 14:23:53 CDT --- (In reply to comment #19)
http://source.winehq.org/git/wine.git/commitdiff/adc0518af74d79e45b68a4119d3...
http://bugs.winehq.org/show_bug.cgi?id=22796
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #21 from Alexandre Julliard julliard@winehq.org 2012-05-11 13:22:01 CDT --- Closing bugs fixed in 1.5.4.
-
wine-bugs＠winehq.org