[Bug 54904] New: RtlInitializeSid should return NTSTATUS
https://bugs.winehq.org/show_bug.cgi?id=54904 Bug ID: 54904 Summary: RtlInitializeSid should return NTSTATUS Product: Wine Version: 8.0 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: ntdll Assignee: wine-bugs(a)winehq.org Reporter: ake.rehnman(a)gmail.com Distribution: --- Created attachment 74410 --> https://bugs.winehq.org/attachment.cgi?id=74410 patch attempt RtlInitializeSid should return NTSTATUS see https://learn.microsoft.com/en-us/windows-hardware/drivers/ddi/ntifs/nf-ntif... -- 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=54904 ake.rehnman(a)gmail.com changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #74410|0 |1 is obsolete| | --- Comment #1 from ake.rehnman(a)gmail.com --- Created attachment 74412 --> https://bugs.winehq.org/attachment.cgi?id=74412 patch attempt 2 Updated comments to reflect NTSTATUS return. -- 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=54904 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Ever confirmed|0 |1 --- Comment #2 from Nikolay Sivov <bunglehead(a)gmail.com> --- In principle this looks going into right direction. I do get STATUS_INVALID_PARAMETER return value on Windows. However, this needs tests obviously. And our count check is incorrect, it should be strict greater-than. -- 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=54904 --- Comment #3 from Nikolay Sivov <bunglehead(a)gmail.com> --- Hi, Ake. Are you going to submit this fix? -- 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=54904 --- Comment #4 from Nikolay Sivov <bunglehead(a)gmail.com> --- MR for this https://gitlab.winehq.org/wine/wine/-/merge_requests/2737 -- 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=54904 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED Fixed by SHA1| |f6e11d656622a9b9aa0b2bd0ea8 | |69dc5ce03c9d2 --- Comment #5 from Nikolay Sivov <bunglehead(a)gmail.com> --- Fixed with f6e11d656622a9b9aa0b2bd0ea869dc5ce03c9d2. -- 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=54904 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #6 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 8.8. -- 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