https://bugs.winehq.org/show_bug.cgi?id=41934 --- Comment #8 from Sagawa <sagawa.aki+winebugs(a)gmail.com> --- (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. [1] https://testbot.winehq.org/JobDetails.pl?Key=27352 -- 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.