http://bugs.winehq.org/show_bug.cgi?id=27468
Summary: QQ International crashes on attempted login Product: Wine Version: unspecified Platform: x86-64 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: dbghelp AssignedTo: wine-bugs@winehq.org ReportedBy: jarrodbagwell@gmail.com
Created an attachment (id=35120) --> (http://bugs.winehq.org/attachment.cgi?id=35120) A terminal dump of what WINE was doing while installing and attempting to login to QQ International
When attempting to login into QQ International, the following occurs: 1. There is no issue with typing in your user ID number. 2. When clicking the cursor into the password field, about 2 seconds later QQ International crashes, generating it's own QQ error code and asking if it can send a debug report to Tencent. 3. QQ can be restarted successfully, but the process repeats itself if you click in the password entry field.
In the attachment I have a dump of the terminal data. The data covers the installation and first attempted login of QQ International.
http://bugs.winehq.org/show_bug.cgi?id=27468
Jerome Leclanche adys.wh@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #35120|Terminal Dump for QQ |output.txt filename|International Bug | Attachment #35120|application/octet-stream |text/plain mime type| |
http://bugs.winehq.org/show_bug.cgi?id=27468
fracting fracting@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |fracting@gmail.com
--- Comment #1 from fracting fracting@gmail.com 2012-01-04 10:38:07 CST --- still in wine-1.3.36-72-g8d66ca1, with http://dl_dir.qq.com/qqfile/qq/QQ2010/QQIntl1.0.exe
sha1sum: 10f67ede6b52f260257d2217800dd2a700332f9d QQIntl1.0.exe
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #2 from fracting fracting@gmail.com 2012-01-04 10:49:04 CST --- For me the crash only happens when I type a correct username and password and try to login. If I try to login with a incorrect password, then QQ International will let me check my password again.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #3 from fracting fracting@gmail.com 2012-01-17 13:10:35 CST --- Hello, retest with wine-1.3.37, and found two crashes, one with riched20 and another with iexplore.
with winetricks riche20 ie7, qq international no longer crash. I'll report separate bugs for riche20 and iexplore.
For the original report, "crash after clicking the cursor into the password field", I think it is fixed in latest Wine. There is no "Version" information in the original report, so I can't test with the broken version of Wine. Jarrod, do you remember which wine version has this problem? Would you mind retest with wine-1.3.37?
Thanks a lot!
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #4 from fracting fracting@gmail.com 2012-01-17 13:58:16 CST --- Hello, reported two bugs:
Bug 29636 - QQ International crash after login (riched20) Bug 29638 - QQ International crash after login (iexplore)
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #5 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-18 00:07:51 CST --- Ok, I re installed using wine 1.3.37 and I get the same results. After installing gdiplus, ie7 (did not install successfully and crashed all over the place) and vc6run I still get the same bug. If I click the cursor into the password field, it will crash out. However, you can still use the pop out keyboard next to the password field, type in your password that way and you can successfully log in.
Hello, reported two bugs:
Bug 29636 - QQ International crash after login (riched20) Bug 29638 - QQ International crash after login (iexplore)
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #6 from fracting fracting@gmail.com 2012-01-18 00:44:52 CST --- (In reply to comment #5)
Ok, I re installed using wine 1.3.37 and I get the same results. After installing gdiplus, ie7 (did not install successfully and crashed all over the place) and vc6run I still get the same bug. If I click the cursor into the password field, it will crash out. However, you can still use the pop out keyboard next to the password field, type in your password that way and you can successfully log in.
Thanks a lot. I still can't reproduce, could you provide some more information?
- Did you clean your wineprefix everytime? If not please remove ~/.wine and try again. - Could you try *without* any native dlls? (don't override gdiplus, vc6run, ie7 and so on) - What's the QQ version? Is there a downloadable url? - Could you try with this one: http://dl_dir.qq.com/qqfile/qq/QQ2010/QQIntl1.0.exe ?
Thank you!
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #7 from fracting fracting@gmail.com 2012-01-18 10:17:48 CST --- Hi Jarrod, is there a crash dialog and a minidump file for this issue? Could you upload the minidump file? Thanks!
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #8 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-19 01:01:15 CST --- Created attachment 38441 --> http://bugs.winehq.org/attachment.cgi?id=38441 QQ mini dump .dmp file totally clean install
This is the .dmp file generated by QQ when the client crashed. This is a totally clean install, no native DLL's used.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #9 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-19 01:02:14 CST --- Created attachment 38442 --> http://bugs.winehq.org/attachment.cgi?id=38442 QQ mini dump .txt file totally clean install
This is the .txt file generated by QQ when the client crashed. This is a totally clean install, no native DLL's used.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #10 from fracting fracting@gmail.com 2012-01-19 03:07:15 CST --- (In reply to comment #8)
Created attachment 38441 [details] QQ mini dump .dmp file totally clean install
This is the .dmp file generated by QQ when the client crashed. This is a totally clean install, no native DLL's used.
Thanks for uploading, but there seems no debug symbols, did you install wine1.3-dbg?
Backtrace: =>0 0x7eb9d9ac in user32 (+0x3d9ac) (0x0033de1c) 1 0x7eb9e0f6 in user32 (+0x3e0f5) (0x0033e17c) 2 0x7ebbf62f in user32 (+0x5f62e) (0x0033e59c)
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #11 from Vitaliy Margolen vitaliy-bugzilla@kievinfo.com 2012-01-19 08:43:48 CST --- Unless you using the same binary Wine package minidumps are useless with Wine. You need to open them on the same machine with the same exact Wine using winedbg.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #12 from fracting fracting@gmail.com 2012-01-19 08:53:22 CST --- (In reply to comment #11)
Unless you using the same binary Wine package minidumps are useless with Wine. You need to open them on the same machine with the same exact Wine using winedbg.
Thanks Vitaliy,and sorry for mislanding :(
Is it a good idea to upload the winedbg output instead of the minidump file?
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #13 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-19 10:01:38 CST --- I still have the .dmp file I think. If someone could tell me or point me to some information on what I need to do with it to make it usable, I'll process it.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #14 from Vitaliy Margolen vitaliy-bugzilla@kievinfo.com 2012-01-19 20:09:10 CST --- winedbg file.dmp
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #15 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-20 00:11:27 CST --- Created attachment 38456 --> http://bugs.winehq.org/attachment.cgi?id=38456 Windbg of QQ .dmp
This is the result of using winedbg on the .dmp file that QQ created upon the crash. This is from a totally clean install of QQInt1.1, no native DLL's installed.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #16 from Qian Hong fracting@gmail.com 2012-01-20 00:47:19 CST --- (In reply to comment #15)
Created attachment 38456 [details] Windbg of QQ .dmp
This is the result of using winedbg on the .dmp file that QQ created upon the crash. This is from a totally clean install of QQInt1.1, no native DLL's installed.
Thanks a lot. BTW, next time please use .txt or .log as extension name for text file :)
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #17 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-20 00:54:13 CST --- Ok, will do. As you have probably guessed this is my first time doing this.
http://bugs.winehq.org/show_bug.cgi?id=27468
Dmitry Timoshkov dmitry@baikal.ru changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #38456|application/x-extension-dmp |text/plain mime type| |
http://bugs.winehq.org/show_bug.cgi?id=27468
Jarrod Bagwell jarrodbagwell@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jarrodbagwell@gmail.com
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #18 from Vitaliy Margolen vitaliy-bugzilla@kievinfo.com 2012-01-20 08:32:13 CST --- Can you attach WINEDEBUG=+tid,+hook,+relay log? Please compress it with gzip/bzip2.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #19 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-20 09:45:45 CST --- (In reply to comment #18)
Can you attach WINEDEBUG=+tid,+hook,+relay log? Please compress it with gzip/bzip2.
I ran the command and my terminal window went nuts spewing stuff. Problem is that QQ never launched. Also, is there a command I can use so that the terminal log is automatically saved into a separate file? I think terminal cut off all of the log so I can't copy/paste the complete thing into a file to upload.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #20 from Qian Hong fracting@gmail.com 2012-01-20 10:12:04 CST --- (In reply to comment #19)
Also, is there a command I can use so that the terminal log is automatically saved into a separate file? I think terminal cut off all of the log so I can't copy/paste the complete thing into a file to upload.
Please try WINEDEBUG=+tid,+hook,+relay wine QQ.exe 2>&1 | tee QQ_crashes_on_attempted_login.log
(cd to ~/.wine/drive_c/Program Files/Tencent/QQIntl/Bin first)
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #21 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-20 21:04:39 CST --- I copy/pasted the command you wrote down after cding into my .wine-qq1.1 directory. It looks like it split what you wrote into two commands. The first made terminal start spewing all the debug data, and QQ didn't open again. The send command, which it picked up after the |tee, read command not found. Was there a mistake in the command that you wrote down for me? Also the log file this will create, where is it going to be placed.?
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #22 from Vitaliy Margolen vitaliy-bugzilla@kievinfo.com 2012-01-20 21:26:36 CST --- Of course command is wrong. Shouldn't use tee with relay log. Use this: WINEDEBUG=+tid,+hook,+relay wine QQ.exe &> QQ_crashes_on_attempted_login.log
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #23 from Vitaliy Margolen vitaliy-bugzilla@kievinfo.com 2012-01-20 21:27:39 CST --- Then run: gzip QQ_crashes_on_attempted_login.log
And attach QQ_crashes_on_attempted_login.log.gz to the bug.
In the future please use forum for these questions. This is bugzilla, not an education channel.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #24 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-20 22:26:11 CST --- (In reply to comment #23)
Then run: gzip QQ_crashes_on_attempted_login.log
And attach QQ_crashes_on_attempted_login.log.gz to the bug.
In the future please use forum for these questions. This is bugzilla, not an education channel.
Thank you for your help Vitaliy. As I said before, I am very new at this and find the wiki page on WINEDEBUG quite confusing to me. I will direct my future inquiries to the forums. I do ask that you have patience though as I am trying to learn all of this to make wine better and more viable for the Chinese base of linux users. In any case, the compressed output is 5.5 mb, so I uploaded it to rapidshare. Here's the download link:
https://rapidshare.com/files/1151599573/QQ_crashes_on_attempted_login.log.gz
http://bugs.winehq.org/show_bug.cgi?id=27468
Vitaliy Margolen vitaliy-bugzilla@kievinfo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|dbghelp |-unknown
--- Comment #25 from Vitaliy Margolen vitaliy-bugzilla@kievinfo.com 2012-01-24 23:27:49 CST --- This looks suspicious: 0040:Call user32.CallNextHookEx(000200fc,00000000,00000100,0033e334) ret=0019506b 0040:trace:hook:call_hook calling hook in thread 0009 WH_KEYBOARD_LL code 0 wp 100 lp 33e334 0040:Ret user32.CallNextHookEx() retval=00000000 ret=0019506b 0040:Ret hook proc 0x194cc3 (id=WH_KEYBOARD_LL,code=0,wp=00000100,lp=0033e334) retval=00000000 0040:Call KERNEL32.GetLastError() ret=78132dba 0040:Ret KERNEL32.GetLastError() retval=000005a4 ret=78132dba
Your log does not contain any records of thread 9. You sure Wine was not running (there were no wineserver process) when you made this log?
0x5a4 = 1444 = ERROR_INVALID_THREAD_ID
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #26 from Jarrod Bagwell jarrodbagwell@gmail.com 2012-01-24 23:34:41 CST --- I use wine for several programs, but as far as I know none of them were running when I made this bugtrace for you. This trace was done in it's own bottle and it should have been the only thing running.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #27 from Qian Hong fracting@gmail.com 2012-02-03 01:07:52 CST --- Hello, I found something interesting:
- I can reproduce this bug with Ubuntu's wine-1.3.37 and wine-1.4-rc1 - I can't reproduce this bug with my wine build from git source, same version number
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #28 from Qian Hong fracting@gmail.com 2012-02-03 06:16:32 CST --- Hello, retest with latest git:
This bug can be reproduced if wine is builded with CFLAGS="-O2" or CFLAGS="-O1", CFLAGS="-O2" is used by the Ubuntu packager. I can't reproduce the bug if wine is builded with CFLAGS="-O0"
wine-1.4-rc2
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #29 from Qian Hong fracting@gmail.com 2012-02-06 10:53:33 CST --- (In reply to comment #28)
Hello, retest with latest git:
This bug can be reproduced if wine is builded with CFLAGS="-O2" or CFLAGS="-O1", CFLAGS="-O2" is used by the Ubuntu packager. I can't reproduce the bug if wine is builded with CFLAGS="-O0"
wine-1.4-rc2
Hello, QQ.exe try to call call_hook_proc() function when clicking on the password form, however, call_hook_proc in hook.c is missing when CFLAGS="-g -O2"
--- snip --- WineDbg starting on pid 0023 start_process () at /home/fracting/wine-git/dlls/kernel32/process.c:1083 0x7ed80e53 start_process+0x53 [/home/fracting/wine-git/dlls/kernel32/process.c:1083] in kernel32: movl %edi,0x4(%esp) 1083 return call_process_entry( peb, entry ); Wine-dbg>b call_hook_proc fixme:dbghelp_dwarf:dwarf2_parse_variable Unsupported constant (parameter) pEncodePara in function fixme:dbghelp_dwarf:dwarf2_parse_variable Unsupported constant (parameter) dwFlags in function fixme:dbghelp_dwarf:compute_location Unhandled attr op: 9d fixme:dbghelp_dwarf:compute_location Unhandled attr op: 9d fixme:dbghelp_dwarf:compute_location Unhandled attr op: 9e fixme:dbghelp_dwarf:compute_location Unhandled attr op: 9e fixme:dbghelp_dwarf:compute_location Only supporting one reg (edi/24 -> -2) No symbols found for call_hook_proc Unable to add breakpoint, will check again when a new DLL is loaded --- snip ---
A dirty workaround is removing the "static" before call_hook_proc, another way of workaround is adding below line in the call_hook() function: TRACE("call_hook_proc is %p\n", call_hook_proc);
With any of the workaround, I can get the call_hook_proc symbol in winedbg again, and the crash disappear.
I don't have enough knowledge to fix this bug, is it a bug of winegcc or a bug of hook.c? Great appreciating if anyone take this bug :)
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #30 from Qian Hong fracting@gmail.com 2012-03-01 20:16:26 CST --- (In reply to comment #28)
Hello, retest with latest git:
This bug can be reproduced if wine is builded with CFLAGS="-O2" or CFLAGS="-O1", CFLAGS="-O2" is used by the Ubuntu packager. I can't reproduce the bug if wine is builded with CFLAGS="-O0"
wine-1.4-rc2
There is an error in my previous description. In fact, CFLAGS="-O0" and CFLAGS="-O1" is ok but CFLAGS="-O2" will cause the crashing.
http://bugs.winehq.org/show_bug.cgi?id=27468
Aric Stewart aric@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |aric@codeweavers.com
--- Comment #31 from Aric Stewart aric@codeweavers.com 2012-04-20 06:58:15 CDT --- What happens if you turn on +relay?
do you see the "Call hook proc" message?
it could be that call_hook_proc is being optimized into inline so the function would vanish but not necessary not be being used.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #32 from Qian Hong fracting@gmail.com 2012-04-20 10:33:21 CDT --- (In reply to comment #31)
What happens if you turn on +relay?
do you see the "Call hook proc" message?
it could be that call_hook_proc is being optimized into inline so the function would vanish but not necessary not be being used.
I can't test with wine-1.5.2-144-gbd3a941 any more, QQ.exe couldn't start, reporting "Initialization failure: 0x00000004"
Will do a bisect test and report a new bug.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #33 from Qian Hong fracting@gmail.com 2012-04-20 11:08:56 CDT --- (In reply to comment #32)
(In reply to comment #31)
What happens if you turn on +relay?
do you see the "Call hook proc" message?
it could be that call_hook_proc is being optimized into inline so the function would vanish but not necessary not be being used.
I can't test with wine-1.5.2-144-gbd3a941 any more, QQ.exe couldn't start, reporting "Initialization failure: 0x00000004"
Will do a bisect test and report a new bug.
I rebuild wine-1.4-rc2, but hit on the "Initialization failure: 0x00000004" again... Maybe broke by some library package upgrading on my Ubuntu 11.10? How to debug in this case?
(Wine ppa's wine-1.4 binary works for me)
http://bugs.winehq.org/show_bug.cgi?id=27468
Ma Xiaojun damage3025@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |damage3025@gmail.com
--- Comment #34 from Ma Xiaojun damage3025@gmail.com 2012-04-26 11:57:27 CDT --- New bug report for "Initialization failure: 0x00000004" http://bugs.winehq.org/show_bug.cgi?id=30512
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #35 from Qian Hong fracting@gmail.com 2012-04-27 12:31:06 CDT --- (In reply to comment #34)
New bug report for "Initialization failure: 0x00000004" http://bugs.winehq.org/show_bug.cgi?id=30512
Thanks Xiaojun, confirming Bug 30512 blocks this bug.
Please ignore Comment 33, I found the reason for the strange bisect result, I didn't delete the stub alt80 when downgrade to older version of Wine.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #36 from Qian Hong fracting@gmail.com 2012-04-27 12:36:54 CDT --- (In reply to comment #31)
What happens if you turn on +relay?
do you see the "Call hook proc" message?
it could be that call_hook_proc is being optimized into inline so the function would vanish but not necessary not be being used.
Hi Aric,
Yes there are "Call hook proc" message, appear twice:
0009:Call hook proc 0x19cf51 (id=WH_KEYBOARD_LL,code=0,wp=00000100,lp=00029:Call KERNEL32.InterlockedDecrement(022ca014) ret=30107bdd
What can I do to help this bug getting fixed?
Thanks.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #37 from Aric Stewart aric@codeweavers.com 2012-04-27 12:50:00 CDT --- (In reply to comment #36)
(In reply to comment #31)
What happens if you turn on +relay?
do you see the "Call hook proc" message?
it could be that call_hook_proc is being optimized into inline so the function would vanish but not necessary not be being used.
Hi Aric,
Yes there are "Call hook proc" message, appear twice:
0009:Call hook proc 0x19cf51 (id=WH_KEYBOARD_LL,code=0,wp=00000100,lp=00029:Call KERNEL32.InterlockedDecrement(022ca014) ret=30107bdd
What can I do to help this bug getting fixed?
Thanks.
So that shows that the call_hook_proc is being optimized into an inline function, so it being "missing" is not the problem.
generate a log with +hook,+relay, both with -O0 and -O2 and look for differences, that may give a clue as to where the failure really is.
-aric
http://bugs.winehq.org/show_bug.cgi?id=27468
johan.gardhage@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |johan.gardhage@gmail.com
http://bugs.winehq.org/show_bug.cgi?id=27468
lizhenbo litimetal@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |litimetal@gmail.com
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #38 from lizhenbo litimetal@gmail.com 2013-01-31 08:40:55 CST --- (In reply to comment #1)
still in wine-1.3.36-72-g8d66ca1, with http://dl_dir.qq.com/qqfile/qq/QQ2010/QQIntl1.0.exe
sha1sum: 10f67ede6b52f260257d2217800dd2a700332f9d QQIntl1.0.exe
This is obsoleted by Tencent. Could you please offer a new link? Thanks.
http://bugs.winehq.org/show_bug.cgi?id=27468
Qian Hong fracting@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download Status|UNCONFIRMED |NEW URL| |http://dl_dir.qq.com/qqfile | |/qq/QQ2012/QQIntl1.6.exe Ever Confirmed|0 |1
--- Comment #39 from Qian Hong fracting@gmail.com 2013-01-31 11:00:18 CST --- Hello, still present with wine-1.5.22-254-gc14bdaf + QQIntl1.6.exe
$ sha1sum QQIntl1.6.exe 7a9f585b4d54c3b69fb1049570d9fc59ba25477a QQIntl1.6.exe
Updated the URL to the latest version of QQIntl
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #40 from lizhenbo litimetal@gmail.com 2013-01-31 19:38:49 CST --- Created attachment 43400 --> http://bugs.winehq.org/attachment.cgi?id=43400 screenshot
$ wine --version wine-1.5.22-254-gc14bdaf build with $ gcc --version gcc (GCC) 4.5.4 ./configure CFLAGS=""-g -O0" CC="gcc" CXX="g++"
I can successfully login, but some problem still exists.
BTW, QQ will crash few minutes later after I logged in.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #41 from lizhenbo litimetal@gmail.com 2013-02-02 23:54:46 CST --- Created attachment 43427 --> http://bugs.winehq.org/attachment.cgi?id=43427 backtrace
I re-build my wine-1.5.22 with -O2 This time, QQ crashed when I was typing password
http://bugs.winehq.org/show_bug.cgi?id=27468
Jactry Zeng jactry92@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jactry92@gmail.com
--- Comment #42 from Jactry Zeng jactry92@gmail.com 2013-02-06 02:22:48 CST --- Also affects to QTalk. sha1sum QT3.5.4.6384.exe 605171c34a3103aaa4664e559f86b7bd022c78d9 QT3.5.4.6384.exe http://dldir1.qq.com/qt/QT3.6.4.6625.exe
http://bugs.winehq.org/show_bug.cgi?id=27468
Jactry Zeng jactry92@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |33076
http://bugs.winehq.org/show_bug.cgi?id=27468
Qian Hong fracting@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |fangendoucg@gmail.com
--- Comment #43 from Qian Hong fracting@gmail.com 2013-04-18 04:32:04 CDT --- *** Bug 33412 has been marked as a duplicate of this bug. ***
http://bugs.winehq.org/show_bug.cgi?id=27468
Qian Hong fracting@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|QQ International crashes on |QQ International crashes |attempted login |when clicking on password | |form if dlls/user32/hook.c | |is build with -O2
--- Comment #44 from Qian Hong fracting@gmail.com 2013-04-18 04:34:45 CDT --- Hello, edited summary to avoid duplicate bug report.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #45 from Dan Kegel dank@kegel.com 2013-04-18 23:53:28 CDT --- Created attachment 44214 --> http://bugs.winehq.org/attachment.cgi?id=44214 Draft patch to disable inlining of that function
Does this patch help?
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #46 from fangendou fangendoucg@gmail.com 2013-04-19 00:36:43 CDT --- (In reply to comment #45)
Created attachment 44214 [details] Draft patch to disable inlining of that function
Does this patch help?
Thanks Dan. It works in my wine-1.5.28 and QQ International 1.6. There's no crash while clicking on and entering password.
--- Comment #47 from fangendou fangendoucg@gmail.com 2013-04-19 00:36:44 CDT --- (In reply to comment #45)
Created attachment 44214 [details] Draft patch to disable inlining of that function
Does this patch help?
Thanks Dan. It works in my wine-1.5.28 and QQ International 1.6. There's no crash while clicking on and entering password.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #46 from fangendou fangendoucg@gmail.com 2013-04-19 00:36:43 CDT --- (In reply to comment #45)
Created attachment 44214 [details] Draft patch to disable inlining of that function
Does this patch help?
Thanks Dan. It works in my wine-1.5.28 and QQ International 1.6. There's no crash while clicking on and entering password.
--- Comment #47 from fangendou fangendoucg@gmail.com 2013-04-19 00:36:44 CDT --- (In reply to comment #45)
Created attachment 44214 [details] Draft patch to disable inlining of that function
Does this patch help?
Thanks Dan. It works in my wine-1.5.28 and QQ International 1.6. There's no crash while clicking on and entering password.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #48 from Jactry Zeng jactry92@gmail.com 2013-04-19 01:57:30 CDT --- (In reply to comment #45)
Created attachment 44214 [details] Draft patch to disable inlining of that function
Does this patch help?
Thanks Dan. The patch also work for me.
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #49 from lizhenbo litimetal@gmail.com 2013-04-19 07:30:49 CDT --- Thanks, it works for me.
http://bugs.winehq.org/show_bug.cgi?id=27468
Dan Kegel dank@kegel.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #44214|0 |1 is obsolete| |
--- Comment #50 from Dan Kegel dank@kegel.com 2013-04-20 00:01:44 CDT --- Created attachment 44223 --> http://bugs.winehq.org/attachment.cgi?id=44223 Wrap hook proc in case it uses wrong calling convention
Jacek suggests it could be a calling convention mismatch... and sure enough, the address of the local variable ret in call_hook_proc changes. As Jacek suggests, the thing to do is insulate ourselves from the value of SP on return from the proc, as in WINPROC_wrapper(). This patch works for me with QQ International on 32 bits. Does it work for you, too?
http://bugs.winehq.org/show_bug.cgi?id=27468
--- Comment #51 from Qian Hong fracting@gmail.com 2013-04-20 00:44:17 CDT --- (In reply to comment #50)
Created attachment 44223 [details] Wrap hook proc in case it uses wrong calling convention
Jacek suggests it could be a calling convention mismatch... and sure enough, the address of the local variable ret in call_hook_proc changes. As Jacek suggests, the thing to do is insulate ourselves from the value of SP on return from the proc, as in WINPROC_wrapper(). This patch works for me with QQ International on 32 bits. Does it work for you, too?
Hi Dan,
This patch works for me, thanks a lot, great work! Thanks Jacek as well, as always.
http://bugs.winehq.org/show_bug.cgi?id=27468
Qian Hong fracting@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |c9d1a1906e9342a707a63d73b4a | |fffcc0577ed11 Status|NEW |RESOLVED Component|-unknown |user32 Resolution| |FIXED
--- Comment #52 from Qian Hong fracting@gmail.com 2013-04-24 23:48:12 CDT --- Fixed by http://source.winehq.org/git/wine.git/commit/c9d1a1906e9342a707a63d73b4afffc...
Thanks for the bug report, thanks Dan, and thanks everyone helped!
http://bugs.winehq.org/show_bug.cgi?id=27468
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #53 from Alexandre Julliard julliard@winehq.org 2013-04-26 13:15:17 CDT --- Closing bugs fixed in 1.5.29.