[Bug 51213] New: GZDoom Builder 2.3 r2787 fails with IdentityNotMappedException
https://bugs.winehq.org/show_bug.cgi?id=51213 Bug ID: 51213 Summary: GZDoom Builder 2.3 r2787 fails with IdentityNotMappedException Product: Wine Version: 6.9 Hardware: x86-64 OS: Linux Status: NEW Severity: normal Priority: P2 Component: mscoree Assignee: wine-bugs(a)winehq.org Reporter: madewokherd(a)gmail.com Distribution: --- Tested using Wine 6.9 and Wine Mono 6.2.0. GZDoom Builder 2.3 r2787 fails to start with an exception: [00000024:] EXCEPTION handling: System.Security.Principal.IdentityNotMappedException: Unable to map SID: S-1-5-21-0-0-0-1000 at System.Security.Principal.SecurityIdentifier.Translate (System.Type) [0x0004d] in <db04fe9e4bca4610adf3414d35e3c1a4>:0 at System.Security.AccessControl.ObjectSecurity.InternalAccessRuleFactory (System.Security.AccessControl.QualifiedAce,System.Type,System.Security.AccessControl.AccessControlType) [0x00007] in <db04fe9e4bca4610adf3414d35e3c1a4>:0 at System.Security.AccessControl.ObjectSecurity.InternalGetAccessRules (bool,bool,System.Type) [0x00061] in <db04fe9e4bca4610adf3414d35e3c1a4>:0 at System.Security.AccessControl.CommonObjectSecurity.GetAccessRules (bool,bool,System.Type) [0x00000] in <db04fe9e4bca4610adf3414d35e3c1a4>:0 at CodeImp.DoomBuilder.General.CheckWritePremissions (string) [0x0000e] in <1d2596b690724749a5f28b323fe25637>:0 at CodeImp.DoomBuilder.UpdateChecker.PerformCheck (bool) [0x00028] in <1d2596b690724749a5f28b323fe25637>:0 at CodeImp.DoomBuilder.General.Main (string[]) [0x0060c] in <1d2596b690724749a5f28b323fe25637>:0 at (wrapper runtime-invoke) <Module>.runtime_invoke_void_object (object,intptr,intptr,intptr) [0x00027] in <1d2596b690724749a5f28b323fe25637>:0 It seems like Mono's version of the Translate function can only handle well-known SIDs, not a user SID. -- 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=51213 Esme Povirk <madewokherd(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download URL| |https://web.archive.org/web | |/20170113205844/http://devb | |uilds.drdteam.org/doombuild | |er2-gzdb/GZDoom_Builder-r27 | |87.7z -- 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=51213 --- Comment #1 from Esme Povirk <madewokherd(a)gmail.com> --- We may want to try importing the corefx version of these classes for Wine Mono, it would also help with bug 25167. -- 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=51213 --- Comment #2 from Esme Povirk <madewokherd(a)gmail.com> --- I pushed the import here: https://github.com/madewokherd/mono/tree/principal Due to bugs/gaps in Wine's implementation of functions that these classes use, I can't commit the import without introducing regressions. -- 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=51213 --- Comment #3 from Esme Povirk <madewokherd(a)gmail.com> --- Import pushed to develop branch, CI build here: https://github.com/madewokherd/wine-mono/actions/runs/958225784 I now get a new error: 0024:err:ntdll:NtQueryInformationToken Unhandled token information class 37 [00000024:] EXCEPTION handling: System.Security.SecurityException: GetTokenInformation returned error 1 "Main Application" tid=00000024 this=027a0120 , thread handle : 00b54ab0, state : not waiting at System.Security.Principal.WindowsIdentity.GetTokenInformation (Microsoft.Win32.SafeHandles.SafeAccessTokenHandle,System.Security.Principal.TokenInformationClass,bool) [0x000ba] in <e465ed4e7813490db5d14a98f3174d0e>:0 at System.Security.Principal.WindowsIdentity.AddDeviceGroupSidClaims (System.Collections.Generic.List`1<System.Security.Claims.Claim>,System.Security.Principal.TokenInformationClass) [0x00014] in <e465ed4e7813490db5d14a98f3174d0e>:0 at System.Security.Principal.WindowsIdentity.InitializeClaims () [0x0009b] in <e465ed4e7813490db5d14a98f3174d0e>:0 at System.Security.Principal.WindowsIdentity/<get_Claims>d__92.MoveNext () [0x0003c] in <e465ed4e7813490db5d14a98f3174d0e>:0 at System.Security.Claims.ClaimsIdentity.HasClaim (string,string) [0x00059] in <e465ed4e7813490db5d14a98f3174d0e>:0 at System.Security.Claims.ClaimsPrincipal.IsInRole (string) [0x0002f] in <e465ed4e7813490db5d14a98f3174d0e>:0 at System.Security.Principal.WindowsPrincipal.IsInRole (string) [0x00051] in <e465ed4e7813490db5d14a98f3174d0e>:0 at CodeImp.DoomBuilder.General.CheckWritePremissions (string) [0x00087] in <1d2596b690724749a5f28b323fe25637>:0 at CodeImp.DoomBuilder.UpdateChecker.PerformCheck (bool) [0x00028] in <1d2596b690724749a5f28b323fe25637>:0 at CodeImp.DoomBuilder.General.Main (string[]) [0x0060c] in <1d2596b690724749a5f28b323fe25637>:0 at (wrapper runtime-invoke) <Module>.runtime_invoke_void_object (object,intptr,intptr,intptr) [0x00027] in <1d2596b690724749a5f28b323fe25637>:0 37 is TokenDeviceGroups. It also expects TokenUserClaimAttributes and TokenDeviceClaimAttributes. -- 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=51213 --- Comment #4 from Esme Povirk <madewokherd(a)gmail.com> --- The corefx code accounts for failure in those calls, but it expects an "invalid parameter" code, not an "unimplemented function" code which Wine returns. Treating Wine's failure the same way works around the exception, but I don't think it works correctly, because the program incorrectly reports it doesn't have write access to its location. -- 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=51213 --- Comment #5 from Esme Povirk <madewokherd(a)gmail.com> --- Pushed work-around: https://github.com/madewokherd/mono/commit/a51dd42fb51885f84105d25863a1c1cb8... -- 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=51213 --- Comment #6 from Esme Povirk <madewokherd(a)gmail.com> --- CI build with work-around: https://github.com/madewokherd/wine-mono/actions/runs/962057724 -- 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=51213 Esme Povirk <madewokherd(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED Fixed by SHA1| |451a54bc7a77b8b816f28ad1c61 | |5745e650586ae --- Comment #7 from Esme Povirk <madewokherd(a)gmail.com> --- Fixed in Wine Mono 6.3.0. -- 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=51213 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 6.14. -- 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