[Bug 38090] New: Managed (.NET) debugger fails to load symbol files ('GetFileVersionInfoSizeExW' is a stub)
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(a)winehq.org Reporter: focht(a)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(a)gmail.com> Sat, 20 Dec 2014 19:39:54 +0000 (13:39 -0600) committer Alexandre Julliard <julliard(a)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 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Anastasius Focht <focht(a)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 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|regression | Regression SHA1|caaf6aafa96d1f8ac72f5e019c4 | |c97ef3d1958e6 | --- Comment #1 from Austin English <austinenglish(a)gmail.com> --- Not technically a regression. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Dmitry Timoshkov <dmitry(a)baikal.ru> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression --- Comment #2 from Dmitry Timoshkov <dmitry(a)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. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |austinenglish(a)gmail.com Component|-unknown |version -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 --- Comment #3 from Austin English <austinenglish(a)gmail.com> --- Created attachment 50761 --> https://bugs.winehq.org/attachment.cgi?id=50761 patch Does this help? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 --- Comment #4 from Hans Leidekker <hans(a)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). -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #50761|0 |1 is obsolete| | --- Comment #5 from Austin English <austinenglish(a)gmail.com> --- Created attachment 50763 --> https://bugs.winehq.org/attachment.cgi?id=50763 try 2 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 --- Comment #6 from Austin English <austinenglish(a)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 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 --- Comment #7 from Hans Leidekker <hans(a)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. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #50763|0 |1 is obsolete| | --- Comment #8 from Austin English <austinenglish(a)gmail.com> --- Created attachment 50764 --> https://bugs.winehq.org/attachment.cgi?id=50764 try 3 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 --- Comment #9 from Anastasius Focht <focht(a)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(a)gmail.com> Tue, 23 Dec 2014 01:37:46 +0000 (19:37 -0600) committer Alexandre Julliard <julliard(a)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 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 --- Comment #10 from Austin English <austinenglish(a)gmail.com> --- https://source.winehq.org/patches/data/109177 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Regression SHA1| |caaf6aafa96d1f8ac72f5e019c4 | |c97ef3d1958e6 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 --- Comment #11 from Anastasius Focht <focht(a)gmx.net> --- Hello folks, obviously still present. $ wine --version wine-1.7.40-29-gc1c108f Regards -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #50764|0 |1 is obsolete| | --- Comment #12 from Austin English <austinenglish(a)gmail.com> --- Created attachment 51525 --> https://bugs.winehq.org/attachment.cgi?id=51525 try 4 Does this help? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 --- Comment #13 from Austin English <austinenglish(a)gmail.com> --- https://source.winehq.org/git/wine.git/commitdiff/e3ae89e8e204eb7948f7edfb86... -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |e3ae89e8e204eb7948f7edfb869 | |d171acdb71a68 Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #14 from Anastasius Focht <focht(a)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 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #15 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 1.7.44. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=38090 Anastasius Focht <focht(a)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 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (2)
-
wine-bugs＠winehq.org -
WineHQ Bugzilla