https://bugs.winehq.org/show_bug.cgi?id=41934
--- Comment #9 from Dmitry Timoshkov dmitry@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