[Bug 47857] New: Logos Bible Software (.Net 4.7.2 app) crashes in win8/10 (IsNormalizedString stub sets error incorrectly apparently)
https://bugs.winehq.org/show_bug.cgi?id=47857 Bug ID: 47857 Summary: Logos Bible Software (.Net 4.7.2 app) crashes in win8/10 (IsNormalizedString stub sets error incorrectly apparently) Product: Wine Version: 4.17 Hardware: x86 OS: Linux Status: NEW Severity: normal Priority: P2 Component: kernel32 Assignee: wine-bugs(a)winehq.org Reporter: xerox.xerox2000x(a)gmail.com Distribution: --- Created attachment 65360 --> https://bugs.winehq.org/attachment.cgi?id=65360 suggested patch After working around bug 47856 it runs into next crash: 0094:fixme:nls:IsNormalizedString 1 0x3563d0c 16 00a0:fixme:ver:GetCurrentPackageId (0x21af2f30 (nil)): stub 009b:fixme:path:parse_url failed to parse L"System.Core" 009b:fixme:combase:RoGetParameterizedTypeInstanceIID stub: 4 0000000029B0EDF0 000000001E79B310 000000001E79BD00 0000000000000000 009b:fixme:path:parse_url failed to parse L"System.Xml" 009b:fixme:path:parse_url failed to parse L"System.Data" 00a0:fixme:winsock:convert_aiflag_w2u Unhandled windows AI_xxx flags 0x20000 00a0:fixme:advapi:RegisterEventSourceW ((null),L".NET Runtime"): stub 00a0:fixme:advapi:ReportEventW (0xcafe4242,0x0001,0x0000,0x00000401,(nil),0x0001,0x00000000,0x21af4560,(nil)): stub 00a0:err:eventlog:ReportEventW L"Application: Logos.exe\nFramework Version: v4.0.30319\nDescription: The application requested process termination through System.Environment.FailFast(string message).\nMessage: An exception occurred in AppModel.Run\nStack:\n at System.Environment.FailFast(System.String, System.Exception"... 00a0:fixme:advapi:DeregisterEventSource (0xcafe4242) stub The IsNormalizedstring fixme seems the culprit; first i hacked the stub just to return true and that worked, but it is also enough to remove the SetLasterror(ERROR_CALL_NOT_IMPLEMENTED) from the stub. https://docs.microsoft.com/en-us/windows/win32/api/winnls/nf-winnls-isnormal... suggests to check for last error ERROR_SUCCESS if the function returns FALSE, so i guess that would be a more logical way of setting that error ( or maybe set no error at all???). I'll attach suggested patch hereafter. After this the program starts successfully for me. -- 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=47857 --- Comment #1 from Nikolay Sivov <bunglehead(a)gmail.com> --- Since we have normalization partially implemented, naive implementation could do normalize + compare check. That would be better than error code manipulations. -- 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=47857 Louis Lenders <xerox.xerox2000x(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Logos Bible Software (.Net |Logos Bible Software (.Net |4.7.2 app) crashes in |4.7.2 app) crashes in |win8/10 (IsNormalizedString |win8/10 (because |stub sets error incorrectly |IsNormalizedString is just |apparently) |a stub) --- Comment #2 from Louis Lenders <xerox.xerox2000x(a)gmail.com> --- (In reply to Nikolay Sivov from comment #1)
Since we have normalization partially implemented, naive implementation could do normalize + compare check. That would be better than error code manipulations.
Ok, I change title of bug then -- 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=47857 Louis Lenders <xerox.xerox2000x(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- URL| |https://downloads.logoscdn. | |com/LBS8/Installer/8.7.0.00 | |39/Logos-x86.msi Keywords| |dotnet, download --- Comment #3 from Louis Lenders <xerox.xerox2000x(a)gmail.com> --- add some info -- 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=47857 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dark.shadow4(a)web.de -- 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=47857 Don Pobanz <dpobanz(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dpobanz(a)gmail.com -- 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=47857 --- Comment #4 from Louis Lenders <xerox.xerox2000x(a)gmail.com> --- Created attachment 66003 --> https://bugs.winehq.org/attachment.cgi?id=66003 updated hack to current git Still present in current wine-(staging)5.0rc; updated hack against current git. With hack Logos starts fine in win8/win10 mode -- 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=47857 Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch -- 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=47857 Daniel Ribeiro da Silva <ferion11(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ferion11(a)hotmail.com --- Comment #5 from Daniel Ribeiro da Silva <ferion11(a)hotmail.com> --- Just for note: There is a very good, stable, opensource (MIT License) implementation of Normalization strings on: https://juliastrings.github.io/utf8proc/ -- 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=47857 --- Comment #6 from Alexandre Julliard <julliard(a)winehq.org> --- IsNormalizedString is implemented now, please retest. -- 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=47857 Louis Lenders <xerox.xerox2000x(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |3e6b15c7b5234002b0af2720df5 | |8cdf084cba40c Resolution|--- |FIXED Status|NEW |RESOLVED --- Comment #7 from Louis Lenders <xerox.xerox2000x(a)gmail.com> --- Yes it`s fixed, thanks! In git from thursday: 0038:err:eventlog:ReportEventW L"Application: Logos.exe\nFramework Version: v4.0.30319\nDescription: The application requested process termination through System.Environment.FailFast(string message).\nMessage: An exception occurred in AppModel.Run\nStack:\n at System.Environment.FailFast(System.String, System.Exception"... In today`s git: Starts fine now (when version set to win8/10.) -- 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=47857 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #8 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 5.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=47857 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |focht(a)gmx.net Summary|Logos Bible Software (.Net |Logos 8 Bible Software |4.7.2 app) crashes in |(.Net 4.7.2 app) crashes in |win8/10 (because |win8/10 (because |IsNormalizedString is just |IsNormalizedString is just |a stub) |a stub) URL|https://downloads.logoscdn. |https://web.archive.org/web |com/LBS8/Installer/8.7.0.00 |/20210209172851/https://dow |39/Logos-x86.msi |nloads.logoscdn.com/LBS8/In | |staller/8.7.0.0039/Logos-x8 | |6.msi -- 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 (1)
-
WineHQ Bugzilla