http://bugs.winehq.org/show_bug.cgi?id=46555 Bernhard Übelacker <bernhardu@mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bernhardu@mailbox.org --- Comment #3 from Bernhard Übelacker <bernhardu@mailbox.org> --- Created attachment 80147 --> http://bugs.winehq.org/attachment.cgi?id=80147 46555_cygwin_AddAccessAllowedAceEx.txt Still present in wine-11.0-rc5. This are the steps that lead to this logging by the installer: - [1] calls SetTokenInformation with TokenOwner, which fails because in NtSetInformationToken TokenOwner is not yet implemented. There is also a line "NtSetInformationToken unimplemented class 4" written. - Because of this failure of SetTokenInformation the function setDefaultSecurity is left early and the line [2] in the installer is not reached, which assigns the global variable "groupSID", which therefore stays NULL. - This "groupSID" is used to initialize a local variable in GetPosixPerms [4], which gets used in a call to AddAccessAllowedAceEx, which fails because group_sid is NULL, which fails the RtlValidSid test returning STATUS_INVALID_SID, and therefore causes the logging. [1] https://cygwin.com/cgit/cygwin-apps/setup/tree/win32.cc#n348 [2] https://cygwin.com/cgit/cygwin-apps/setup/tree/win32.cc#n361 [3] https://cygwin.com/cgit/cygwin-apps/setup/tree/win32.cc#n70 [4] https://cygwin.com/cgit/cygwin-apps/setup/tree/win32.cc#n80 Adding another stub block for TokenOwner in NtSetInformationToken, like at the bottom of attached file, avoids the logging of AddAccessAllowedAceEx. Therefore this bug is about SetTokenInformation supporting TokenOwner. -- 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.