https://bugs.winehq.org/show_bug.cgi?id=38090
Bug ID: 38090 Summary: Managed (.NET) debugger fails to load symbol files ('GetFileVersionInfoSizeExW' is a stub) Product: Wine Version: 1.7.36 Hardware: x86 OS: Linux Status: NEW Severity: normal Priority: P2 Component: -unknown Assignee: wine-bugs@winehq.org Reporter: focht@gmx.net Distribution: ---
Hello folks,
found while investigating bugs with managed applications (.NET).
Dump of managed callstacks doesn't work anymore, instead the debugger prints 'Failed to load data access DLL foo, 80004005'.
--- snip --- ... 0009:Call version.GetFileVersionInfoSizeExW(00000000,00438f88 L"z:\home\focht\downloads\debug_symbols\mscorwks.dll\4333E7EC561000\mscordacwks_x86_x86_2.0.50727.42.dll",0033c47c) ret=022d643e 0009:fixme:ver:GetFileVersionInfoSizeExW stub: 0 L"z:\home\focht\downloads\debug_symbols\mscorwks.dll\4333E7EC561000\mscordacwks_x86_x86_2.0.50727.42.dll" 0x33c47c 0009:Ret version.GetFileVersionInfoSizeExW() retval=00000000 ret=022d643e 0009:Call KERNEL32.SetErrorMode(00000001) ret=021e29b4 0009:Ret KERNEL32.SetErrorMode() retval=00000001 ret=021e29b4 0009:Call msvcrt._vsnwprintf(0236af88,00000040,020c65a8 L"Win32 error 0n%d",0033c718) ret=022d4956 0009:Ret msvcrt._vsnwprintf() retval=00000010 ret=022d4956 0009:Call msvcrt._vsnwprintf(0035411c,00000145,0203dbd8 L"ERROR: Unable to load DLL %s, %s",0033c728) ret=022d4956 0009:Ret msvcrt._vsnwprintf() retval=00000050 ret=022d4956 ... 0009:Call KERNEL32.MultiByteToWideChar(00000000,00000000,64273bbc "Failed to load data access DLL, 0x%08x\n",ffffffff,00000000,00000000) ret=020ebf5b 0009:Ret KERNEL32.MultiByteToWideChar() retval=00000028 ret=020ebf5b --- snip ---
It a regression: http://source.winehq.org/git/wine.git/commitdiff/caaf6aafa96d1f8ac72f5e019c4...
--- snip --- author Austin English austinenglish@gmail.com Sat, 20 Dec 2014 19:39:54 +0000 (13:39 -0600)
committer Alexandre Julliard julliard@winehq.org Mon, 22 Dec 2014 18:18:21 +0000 (19:18 +0100)
commit caaf6aafa96d1f8ac72f5e019c4c97ef3d1958e6 tree cec676788e3b8000423aa779ed269408b9804df4 parent ef8db24b9c77bdc110c4c2b6acb4581b9fd6d193
version: Add stubs for GetFileVersionInfoSizeExA/W. --- snip ---
It shouldn't be too hard to add an actual implementation.
Regards
https://bugs.winehq.org/show_bug.cgi?id=38090
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |dotnet, download, | |regression URL| |https://www.microsoft.com/e | |n-us/download/confirmation. | |aspx?id=8279 Regression SHA1| |caaf6aafa96d1f8ac72f5e019c4 | |c97ef3d1958e6
https://bugs.winehq.org/show_bug.cgi?id=38090
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords|regression | Regression SHA1|caaf6aafa96d1f8ac72f5e019c4 | |c97ef3d1958e6 |
--- Comment #1 from Austin English austinenglish@gmail.com --- Not technically a regression.
https://bugs.winehq.org/show_bug.cgi?id=38090
Dmitry Timoshkov dmitry@baikal.ru changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression
--- Comment #2 from Dmitry Timoshkov dmitry@baikal.ru --- (In reply to Austin English from comment #1)
Not technically a regression.
Of course it is, please take the responsibility for your patches, you can't just keep adding the stubs and claim that it's not your competence to fix what broke after that. That should have happened sooner or later, you either start writing real code or stop sending the fakes.
https://bugs.winehq.org/show_bug.cgi?id=38090
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |austinenglish@gmail.com Component|-unknown |version
https://bugs.winehq.org/show_bug.cgi?id=38090
--- Comment #3 from Austin English austinenglish@gmail.com --- Created attachment 50761 --> https://bugs.winehq.org/attachment.cgi?id=50761 patch
Does this help?
https://bugs.winehq.org/show_bug.cgi?id=38090
--- Comment #4 from Hans Leidekker hans@meelstraat.net --- (In reply to Austin English from comment #3)
Created attachment 50761 [details] patch
Does this help?
It would be better to keep the current stub if any flags are set, if not forward to GetFileVersionInfoSize. You don't need the retval variable (and please format flags with %x).
https://bugs.winehq.org/show_bug.cgi?id=38090
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #50761|0 |1 is obsolete| |
--- Comment #5 from Austin English austinenglish@gmail.com --- Created attachment 50763 --> https://bugs.winehq.org/attachment.cgi?id=50763 try 2
https://bugs.winehq.org/show_bug.cgi?id=38090
--- Comment #6 from Austin English austinenglish@gmail.com --- (In reply to Hans Leidekker from comment #4)
(In reply to Austin English from comment #3)
Created attachment 50761 [details] patch
Does this help?
It would be better to keep the current stub if any flags are set, if not forward to GetFileVersionInfoSize. You don't need the retval variable (and please format flags with %x).
Thanks for the feedback Hans. See attachment 50763
https://bugs.winehq.org/show_bug.cgi?id=38090
--- Comment #7 from Hans Leidekker hans@meelstraat.net --- (In reply to Austin English from comment #6)
(In reply to Hans Leidekker from comment #4)
(In reply to Austin English from comment #3)
Created attachment 50761 [details] patch
Does this help?
It would be better to keep the current stub if any flags are set, if not forward to GetFileVersionInfoSize. You don't need the retval variable (and please format flags with %x).
Thanks for the feedback Hans. See attachment 50763 [details]
I'd say one FIXME at the top is sufficient. You don't need the else block because you return from the first branch.
https://bugs.winehq.org/show_bug.cgi?id=38090
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #50763|0 |1 is obsolete| |
--- Comment #8 from Austin English austinenglish@gmail.com --- Created attachment 50764 --> https://bugs.winehq.org/attachment.cgi?id=50764 try 3
https://bugs.winehq.org/show_bug.cgi?id=38090
--- Comment #9 from Anastasius Focht focht@gmx.net --- Hello folks,
the patch helps a bit but lets the CLR debugger immediately run into - guess - GetFileVersionInfoExA/W stubs, committed one day later:
--- snip --- version: Add stubs for GetFileVersionInfoExA/W.
author Austin English austinenglish@gmail.com Tue, 23 Dec 2014 01:37:46 +0000 (19:37 -0600)
committer Alexandre Julliard julliard@winehq.org Tue, 23 Dec 2014 14:30:50 +0000 (15:30 +0100)
commit 61fc9ac92f0dbe80e07d883a2365e16fd6f37774 tree 1b4ace4cee89aff7b8cf0a3119e8677310a9822b parent 0029290e3e769880f2e357cf9608ac26535c29fb --- snip ---
I've created bug 38098 to track this.
Regards
https://bugs.winehq.org/show_bug.cgi?id=38090
--- Comment #10 from Austin English austinenglish@gmail.com --- https://source.winehq.org/patches/data/109177
https://bugs.winehq.org/show_bug.cgi?id=38090
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |caaf6aafa96d1f8ac72f5e019c4 | |c97ef3d1958e6
https://bugs.winehq.org/show_bug.cgi?id=38090
--- Comment #11 from Anastasius Focht focht@gmx.net --- Hello folks,
obviously still present.
$ wine --version wine-1.7.40-29-gc1c108f
Regards
https://bugs.winehq.org/show_bug.cgi?id=38090
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #50764|0 |1 is obsolete| |
--- Comment #12 from Austin English austinenglish@gmail.com --- Created attachment 51525 --> https://bugs.winehq.org/attachment.cgi?id=51525 try 4
Does this help?
https://bugs.winehq.org/show_bug.cgi?id=38090
--- Comment #13 from Austin English austinenglish@gmail.com --- https://source.winehq.org/git/wine.git/commitdiff/e3ae89e8e204eb7948f7edfb86...
https://bugs.winehq.org/show_bug.cgi?id=38090
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |e3ae89e8e204eb7948f7edfb869 | |d171acdb71a68 Status|NEW |RESOLVED Resolution|--- |FIXED
--- Comment #14 from Anastasius Focht focht@gmx.net --- Hello folks,
this is fixed by commit https://source.winehq.org/git/wine.git/commitdiff/e3ae89e8e204eb7948f7edfb86...
Thanks Austin.
Tested managed debugging/callstacks with KeePass 2.29 for Windows (requires .NET Framework 2.0)
http://keepass.info/download.html
--- snip --- .... (2c.36): CLR exception - code e0434f4d (first chance) ... OS Thread Id: 0x36 (7) ESP EIP 0c87e228 7b845511 [HelperMethodFrame: 0c87e228] 0c87e2cc 7a65c1ad System.Diagnostics.PerformanceCounter.Initialize() 0c87e300 7a65ba5e System.Diagnostics.PerformanceCounter.set_RawValue(Int64) 0c87e318 7a5c299b System.Net.NetworkingPerfCounters.Initialize() 0c87e348 7a61f893 System.Net.Configuration.SettingsSectionInternal..ctor(System.Net.Configuration.SettingsSection) 0c87e360 7a61f935 System.Net.Configuration.SettingsSectionInternal.get_Section() 0c87e38c 7a57aaf1 System.Net.HttpWebRequest.get_DefaultMaximumResponseHeadersLength() 0c87e390 7a580a2b System.Net.HttpWebRequest..ctor(System.Uri, System.Net.ServicePoint) 0c87e3b0 7a586457 System.Net.HttpRequestCreator.Create(System.Uri) 0c87e3bc 7a5662be System.Net.WebRequest.Create(System.Uri, Boolean) 0c87e3e0 7a5663ad System.Net.WebRequest.Create(System.Uri) 0c87e3e8 7a5934ab System.Net.WebClient.GetWebRequest(System.Uri) 0c87e3f4 0c4bdf2e KeePassLib.Serialization.IOWebClient.GetWebRequest(System.Uri) 0c87e3fc 7a593cee System.Net.WebClient.OpenRead(System.Uri) 0c87e43c 0c4b04eb KeePassLib.Serialization.IOConnection.OpenRead(KeePassLib.Serialization.IOConnectionInfo) 0c87e44c 0c4b00e3 KeePass.Util.UpdateCheckEx.LoadInfoFile(System.String) 0c87e47c 0c50ff16 KeePass.Util.UpdateCheckEx.DownloadInfoFile(System.Object) ... --- snip ---
Regards
https://bugs.winehq.org/show_bug.cgi?id=38090
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #15 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 1.7.44.
https://bugs.winehq.org/show_bug.cgi?id=38090
Anastasius Focht focht@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- URL|https://www.microsoft.com/e |https://web.archive.org/web |n-us/download/confirmation. |/20110805064614/http://down |aspx?id=8279 |load.microsoft.com/download | |/A/6/A/A6AC035D-DA3F-4F0C-A | |DA4-37C8E5D34E3D/winsdk_web | |.exe