https://bugs.winehq.org/show_bug.cgi?id=41934 --- Comment #9 from Dmitry Timoshkov <dmitry(a)baikal.ru> --- (In reply to Sagawa from comment #8)
(In reply to Dmitry Timoshkov from comment #7)
Thanks for the confirmation. Regarding the patch - I'm not sure whether an existing WinBuiltinAnyPackageSid should be converted to the "AC" alias, or (like my patch does) new WinApplicationPackageAuthoritySid entry should be added to the well-known sid list.
My test [1] based on dlls/advapi32/tests/security.c shows interesting things: - WinApplicationPackageAuthoritySid (=83) doesn't return valid SID. Because win10 outputs 'security.c:2054: Tests skipped: Well known SID 83 not implemented' that means API call is failed. - WinBuiltinAnyPackageSid(=84) returns S-1-15-2-1 as the test passes.
So, the patch needs to be updated before submitting. Sorry for my hasty judgements.
"[MS-DTYP]:Windows Data Types" confirms this: https://msdn.microsoft.com/en-us/library/cc980032.aspx ALL_APP_PACKAGES S-1-15-2-1 and (another page of the same document) https://msdn.microsoft.com/en-us/library/cc230374.aspx "AC" - ALL APP PACKAGES Essentially: "AC" == WinBuiltinAnyPackageSid -- 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.