http://bugs.winehq.org/show_bug.cgi?id=24387
Summary: MS Money 2000: Crashes updating financial informations from the Internet Product: Wine Version: 1.3.2 Platform: x86 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: andrea.vai@unipv.it
Created an attachment (id=30732) --> (http://bugs.winehq.org/attachment.cgi?id=30732) Terminal output.
If I click on "update informations from the internet" in the main page of the application, the applications crashes (Application error window "MSMONEY encountered an error and has must be closed" appears. The only way to close the application is to kill it). Problem was not present in wine 1.1.15, so I performed a regression test and the result is: ---------------------------------------------------------------- c5a6b7b36674e8b7f7dc687b01773b88fa7d7312 is the first bad commit commit c5a6b7b36674e8b7f7dc687b01773b88fa7d7312 Author: Juan Lang juan.lang@gmail.com Date: Tue Mar 30 10:26:48 2010 -0700
rsaenh: Support setting salt via KP_SALT (resend).
:040000 040000 d9a20e02d3637c41baadb3131a23907fb5603fe8 51eea86ad80613f612f64bd44a0517c0b89a4894 M dlls ---------------------------------------------------------------- As far as I can notice, the bad commit is the same involved in bug 23208. Fixing this regression would really bring this app towards complete functionality.
http://bugs.winehq.org/show_bug.cgi?id=24387
Andrea Vai andrea.vai@unipv.it changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression
http://bugs.winehq.org/show_bug.cgi?id=24387
Andrea Vai andrea.vai@unipv.it changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |juan_lang@yahoo.com
http://bugs.winehq.org/show_bug.cgi?id=24387
Andrea Vai andrea.vai@unipv.it changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |andrea.vai@unipv.it
http://bugs.winehq.org/show_bug.cgi?id=24387
Juan Lang juan_lang@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Component|-unknown |rsaenh
--- Comment #1 from Juan Lang juan_lang@yahoo.com 2010-09-13 16:30:17 CDT --- Does reverting that commit fix the crash? If so, please attach a +crypt log.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #2 from Andrea Vai andrea.vai@unipv.it 2010-09-13 17:39:33 CDT --- (In reply to comment #1)
Does reverting that commit fix the crash? If so, please attach a +crypt log.
yes, I reverted the commit and the crash doesn't happen. Please tell me how to produce a +crypt log.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #3 from Juan Lang juan_lang@yahoo.com 2010-09-13 17:41:31 CDT --- http://wiki.winehq.org/FAQ#head-16da35b6327024d6ea576e3678488b16862d0f5e
Replace +relay,+seh,+tid in the example with +crypt.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #4 from Andrea Vai andrea.vai@unipv.it 2010-09-13 18:04:41 CDT --- Created an attachment (id=30737) --> (http://bugs.winehq.org/attachment.cgi?id=30737) +Cc
http://bugs.winehq.org/show_bug.cgi?id=24387
Andrea Vai andrea.vai@unipv.it changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #30737|+Cc |+crypt Log with the commit description| |*not* reverted
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #5 from Andrea Vai andrea.vai@unipv.it 2010-09-13 18:06:59 CDT --- Created an attachment (id=30738) --> (http://bugs.winehq.org/attachment.cgi?id=30738) +crypt log after having the commit reverted
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #6 from Juan Lang juan_lang@yahoo.com 2010-09-13 18:17:51 CDT --- (In reply to comment #5)
Created an attachment (id=30738)
--> (http://bugs.winehq.org/attachment.cgi?id=30738) [details]
+crypt log after having the commit reverted
Wait, this one also shows a crash: trace:crypt:CryptDecrypt (0x1a0218, 0x0, 1, 00000000, 0x32db30, 0x32d868) wine: Unhandled page fault on read access to 0x0000001c at address 0x29e05801 (thread 0009), starting debugger...
Are you sure you attached the correct log?
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #7 from Andrea Vai andrea.vai@unipv.it 2010-09-14 07:49:25 CDT --- (In reply to comment #6)
(In reply to comment #5)
Created an attachment (id=30738)
--> (http://bugs.winehq.org/attachment.cgi?id=30738) [details] [details]
+crypt log after having the commit reverted
Wait, this one also shows a crash: trace:crypt:CryptDecrypt (0x1a0218, 0x0, 1, 00000000, 0x32db30, 0x32d868) wine: Unhandled page fault on read access to 0x0000001c at address 0x29e05801 (thread 0009), starting debugger...
Are you sure you attached the correct log?
Yes, I double checked and I am sure that it is the correct log.
I was also aware of the crash in this log (call it crash #2), but it is not "the" crash (call it crash #1) I had reported in the bug description, so I didn't know if it should be better to take care of it, file another bug for crash#2, or whatever else. Crash#2 happens slightly after I launch the application: it pops up an "application crash" window, anyway the app itself keeps running, and if I close that window and ignore the crash, and follow steps reported to reproduce crash#1, crash#1 does not happen. That's why I said "I reverted the commit and the crash doesn't happen.". I don't know if crash#2 deserves being investigated or if we can say that crash#1 doesn't depend on it, so we don't care about it.
Furthermore, the output of the commit command contains an error (says "1 out of 2 hunks FAILED"), and I attach that output.
Last note: if I revert the commit, recompile, rm .wine, launch the setup program of the application, the setup program reports a crash too, but if I ignore it and go on with the setup, it finishes correctly. Should I post some information about this "crash", too???
Hope I didn't make too many mistakes with all of this.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #8 from Andrea Vai andrea.vai@unipv.it 2010-09-14 07:55:01 CDT --- Created an attachment (id=30749) --> (http://bugs.winehq.org/attachment.cgi?id=30749) Revert commit command output
Terminal output of the command git show c5a6b7b36674e8b7f7dc687b01773b88fa7d7312 | patch -p1 -R
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #9 from Andrea Vai andrea.vai@unipv.it 2010-09-14 07:59:34 CDT --- Created an attachment (id=30750) --> (http://bugs.winehq.org/attachment.cgi?id=30750) rsaenh.c.rej
content of dlls/rsaenh/tests/rsaenh.c.rej, mentioned in attachment (id=30749)
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #10 from Juan Lang juan_lang@yahoo.com 2010-09-14 11:35:11 CDT --- (In reply to comment #7)
I was also aware of the crash in this log (call it crash #2), but it is not "the" crash (call it crash #1) I had reported in the bug description, so I didn't know if it should be better to take care of it, file another bug for crash#2, or whatever else.
Ok. This is fine.
Furthermore, the output of the commit command contains an error (says "1 out of 2 hunks FAILED"), and I attach that output.
This failure is unimportant, it only affects the tests. Thanks for attaching the rsaenh.c.rej so I could verify it.
Hope I didn't make too many mistakes with all of this.
Nope, all seems fine.
I have no idea why the crash is occurring, unfortunately. The ironic thing is, this patch was added expressly for MS Money.
Could you attach a +relay,+crypt log of the crash? Perhaps that will shed more light.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #11 from Andrea Vai andrea.vai@unipv.it 2010-09-14 15:30:31 CDT --- Created an attachment (id=30756) --> (http://bugs.winehq.org/attachment.cgi?id=30756) +relay,+crypt log of the crash
gzip splitted file, part 1 of 2
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #12 from Andrea Vai andrea.vai@unipv.it 2010-09-14 15:31:28 CDT --- Created an attachment (id=30757) --> (http://bugs.winehq.org/attachment.cgi?id=30757) +relay,+crypt log of the crash
gzip splitted file, part 2 of 2
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #13 from Andrea Vai andrea.vai@unipv.it 2010-09-14 15:51:00 CDT --- (In reply to comment #10)
I have no idea why the crash is occurring, unfortunately. The ironic thing is, this patch was added expressly for MS Money.
I could add some other info if it can help: MS Money itself indeed works better as an "out of the box" app in wine, because without that patch winetricks/ies4linux are needed to install the app (as also pointed out in Bug 23208). At least, I never managed to install it without winetricks/ies4linux and without the patch. So, I would say that from that point of view the app works better (winetricks/ies4linux are also a mess), but is it worth it if the patch introduces so many bugs? I would say no.
Could you attach a +relay,+crypt log of the crash? Perhaps that will shed more light.
File attached (37 MB uncompressed, 1.6M zipped, 1M+600k chunks) Thank you for your patience.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #14 from Juan Lang juan_lang@yahoo.com 2010-09-15 12:58:32 CDT --- Created an attachment (id=30772) --> (http://bugs.winehq.org/attachment.cgi?id=30772) Patch: Revert c5a6b7b36674e8b7f7dc687b01773b88fa7d7312
This patch reverts the offending commit on latest git. Does it remove the crash? Unfortunately it's probably not correct, since it probably reverts bug 6698.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #15 from Andrea Vai andrea.vai@unipv.it 2010-09-15 15:58:42 CDT --- (In reply to comment #14)
This patch reverts the offending commit on latest git. Does it remove the crash?
Yes, it does. I tested it applying it to a fresh git-wine downloaded today. Crash is present in HEAD, but after adding the patch, the crash doesn't occur anymore. Juan, does it definitely means that this is a regression?
Unfortunately it's probably not correct, since it probably reverts bug 6698.
BTW: I had to use winetricks/ies4linux to install the application in the patched git tree. If I don't use winetricks/ies4linux BEFORE running the setup program, then the bug 6698 occurs when I open a mny file. This is exactly the same behavior I mentioned in comment #13 (indeed, my expression "never managed to install" in comment #13 should be read "never managed to open an mny file without getting into bug 6698", because the installation itself works even without winetricks/ies4linux, but when the program is runned and an mny file is opened, the file does not open and the "money is not installed correctly" message appears).
I don't know if I have to post more detailed information, maybe that's already too much and in the case sorry for that.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #16 from Juan Lang juan_lang@yahoo.com 2010-09-15 16:13:11 CDT --- (In reply to comment #15)
Juan, does it definitely means that this is a regression?
No. See your input below:
BTW: I had to use winetricks/ies4linux to install the application in the patched git tree. If I don't use winetricks/ies4linux BEFORE running the setup program, then the bug 6698 occurs when I open a mny file. This is exactly the same behavior I mentioned in comment #13 (indeed, my expression "never managed to install" in comment #13 should be read "never managed to open an mny file without getting into bug 6698", because the installation itself works even without winetricks/ies4linux, but when the program is runned and an mny file is opened, the file does not open and the "money is not installed correctly" message appears).
Basically what you're saying is that this crash doesn't exist without this patch because you can't get far enough in the application without the patch applied. In that case, this really isn't a regression: the patch appears to allow the app to get further than it did before, and this is really a new crash. I haven't been able to discover any relationship between the code and the crash, at any rate.
On a hunch, I did the following: I used unpatched Wine (latest git), then I installed dcom98 with winetricks. I removed the DLL overrides for ole32 and rpcrt4, leaving only oleaut32. With this workaround in place, it no longer crashes for me. Can you confirm this works for you?
http://bugs.winehq.org/show_bug.cgi?id=24387
Andrea Vai andrea.vai@unipv.it changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords|regression |
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #17 from Andrea Vai andrea.vai@unipv.it 2010-09-15 17:18:08 CDT --- (In reply to comment #16)
Juan, does it definitely means that this is a regression?
No. See your input below:
...
Basically what you're saying is that this crash doesn't exist without this patch because you can't get far enough in the application without the patch applied. In that case, this really isn't a regression: the patch appears to allow the app to get further than it did before, and this is really a new crash.
ok, got it, supposing that here you refer to "this patch" as c5a6b7b36674e8b7f7dc687b01773b88fa7d7312, and not as the patch that reverts it, am I right? Just to be sure I understand
On a hunch, I did the following: I used unpatched Wine (latest git), then I installed dcom98 with winetricks. I removed the DLL overrides for ole32 and rpcrt4, leaving only oleaut32. With this workaround in place, it no longer crashes for me. Can you confirm this works for you?
Yes, it works! Thank you! Does it means something useful for getting the bug fixed?
http://bugs.winehq.org/show_bug.cgi?id=24387
Juan Lang juan_lang@yahoo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Component|rsaenh |oleaut32 Ever Confirmed|0 |1
--- Comment #18 from Juan Lang juan_lang@yahoo.com 2010-09-15 17:28:49 CDT --- (In reply to comment #17)
ok, got it, supposing that here you refer to "this patch" as c5a6b7b36674e8b7f7dc687b01773b88fa7d7312, and not as the patch that reverts it, am I right? Just to be sure I understand
Oops, yes, right you are. Sorry for the confusion.
Yes, it works! Thank you! Does it means something useful for getting the bug fixed?
Good! At least it means we have a better idea where to look. Maybe a +ole,+typelib,+seh,+tid log might be of use, but it's rather outside my area of expertise, so hopefully someone else will take a look.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #19 from Andrea Vai andrea.vai@unipv.it 2010-09-15 17:39:30 CDT --- (In reply to comment #18)
Maybe a +ole,+typelib,+seh,+tid log might be of use
I will post the log asap. Shall I post the log of the "crashing" version, or the one with dcom98 installed? Or maybe both? Thank you
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #20 from Juan Lang juan_lang@yahoo.com 2010-09-15 17:43:09 CDT --- (In reply to comment #19)
I will post the log asap. Shall I post the log of the "crashing" version, or the one with dcom98 installed? Or maybe both? Thank you
The crashing one only, please. (With dcom98 installed, there won't be any output from these channels, as only Wine's ole32/oleaut32 libraries provide these debug channels.)
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #21 from Andrea Vai andrea.vai@unipv.it 2010-09-17 06:41:58 CDT --- Created an attachment (id=30815) --> (http://bugs.winehq.org/attachment.cgi?id=30815) +ole,+typelib,+seh,+tid log
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #22 from butraxz@gmail.com 2013-06-30 08:52:24 CDT --- This ticket has not been updated for over 900 days.
Is this still an issue in wine version 1.6-rc4 or higher or is this to be closed as abandoned ?
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #23 from Andrea Vai andrea.vai@unipv.it 2013-06-30 15:34:40 CDT --- (In reply to comment #22)
This ticket has not been updated for over 900 days.
Is this still an issue in wine version 1.6-rc4 or higher or is this to be closed as abandoned ?
I can try and let know here in the next days.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #24 from Andrea Vai andrea.vai@unipv.it 2013-07-01 14:42:47 CDT --- The bug still exists in wine 1.6-rc4. I attach a backtrace.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #25 from Andrea Vai andrea.vai@unipv.it 2013-07-01 14:43:54 CDT --- Created attachment 45068 --> http://bugs.winehq.org/attachment.cgi?id=45068 Backtrace in wine 1.6-rc4
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #26 from Andrea Vai andrea.vai@unipv.it --- Bug is still present in wine-1.7.18. Backtrace attached.
http://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #27 from Andrea Vai andrea.vai@unipv.it --- Created attachment 49142 --> http://bugs.winehq.org/attachment.cgi?id=49142 backtrace in wine-1.7.18
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #28 from Austin English austinenglish@gmail.com --- (In reply to Andrea Vai from comment #27)
Created attachment 49142 [details] backtrace in wine-1.7.18
Could you install debugging symbols and attach a new backtrace please?
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #29 from Andrea Vai andrea.vai@unipv.it --- (In reply to Austin English from comment #28)
Could you install debugging symbols and attach a new backtrace please?
Ok, how can I do that?
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #30 from Austin English austinenglish@gmail.com --- (In reply to Andrea Vai from comment #29)
(In reply to Austin English from comment #28)
Could you install debugging symbols and attach a new backtrace please?
Ok, how can I do that?
See 'Backtrace Quality' on http://wiki.winehq.org/Backtraces
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #31 from Andrea Vai andrea.vai@unipv.it --- (In reply to Austin English from comment #30)
Could you install debugging symbols and attach a new backtrace please?
Ok, how can I do that?
See 'Backtrace Quality' on http://wiki.winehq.org/Backtraces
Ok thanks, I hope this is what you need... new backtrace attached
https://bugs.winehq.org/show_bug.cgi?id=24387
Andrea Vai andrea.vai@unipv.it changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #45068|0 |1 is obsolete| | Attachment #49142|0 |1 is obsolete| |
--- Comment #32 from Andrea Vai andrea.vai@unipv.it --- Created attachment 49170 --> https://bugs.winehq.org/attachment.cgi?id=49170 Backtrace in 1.7.23
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #33 from Austin English austinenglish@gmail.com --- This is your friendly reminder that there has been no bug activity for over a year. Is this still an issue in current (1.7.51 or newer) wine?
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #34 from Andrea Vai andrea.vai@unipv.it --- I can try and let know here in the next days.
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #35 from Andrea Vai andrea.vai@unipv.it --- Bug is still present in wine-1.7.51
I am sorry, but I am having some problems in attaching a new backtrace: following instructions at http://wiki.winehq.org/Backtraces I find this output from winedbg (*):
Wine-dbg>info process pid threads executable (all id:s are in hex) 0000000e 7 'services.exe' 00000022 3 _ 'winedevice.exe' 0000001b 3 _ 'plugplay.exe' 00000014 4 _ 'winedevice.exe'
and I don't know which pid to use for attaching the process, as none of them seems related to the program that, indeed, is currently running (msmoney.exe).
(*) I now realize that I am (perhaps (**)) using winedbg 1.7.18 (and I can't find a winedbg executable in the compiled 1.7.51 version, except in programs/winedbg/winedbg.exe.so, which gives me segmentation fault if I launch it). Don't know if this matters.
(**) I can't find a command line option of winedbg to get its version, so I can only guess that the version is 1.7.18 because this is the version number reported at the bottom of the page generated by typing "man winedbg". Don't know if this is a correct guess.
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #36 from Bruno Jesus 00cpxxx@gmail.com --- Please attach a new +rsaenh,+crypt from wine 1.7.50 or later.
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #37 from Andrea Vai andrea.vai@unipv.it --- Created attachment 52736 --> https://bugs.winehq.org/attachment.cgi?id=52736 +rsaenh,+crypt log, from wine-1.7.51
Log attached.
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #38 from Bruno Jesus 00cpxxx@gmail.com --- (In reply to Andrea Vai from comment #37)
Created attachment 52736 [details] +rsaenh,+crypt log, from wine-1.7.51
Log attached.
Thanks for the log, as expected there is nothing wrong with the cryptography part. The problem seems to be inside OLE as pointed in comment 18. I cannot help in that area.
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #39 from Andrea Vai andrea.vai@unipv.it --- Still present in 1.8.2
https://bugs.winehq.org/show_bug.cgi?id=24387
Alistair Leslie-Hughes leslie_alistair@hotmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |leslie_alistair@hotmail.com
--- Comment #40 from Alistair Leslie-Hughes leslie_alistair@hotmail.com --- (In reply to Andrea Vai from comment #39)
Still present in 1.8.2
Does wine-staging have the same issue?
https://bugs.winehq.org/show_bug.cgi?id=24387
--- Comment #41 from Andrea Vai andrea.vai@unipv.it --- (In reply to Alistair Leslie-Hughes from comment #40)
(In reply to Andrea Vai from comment #39)
Still present in 1.8.2
Does wine-staging have the same issue?
Yes, I have verified it right now, using wine-2.13 (Staging)