[Bug 55928] New: NtQuerySystemInformation SystemProcessInformation result misaligned
https://bugs.winehq.org/show_bug.cgi?id=55928 Bug ID: 55928 Summary: NtQuerySystemInformation SystemProcessInformation result misaligned Product: Wine Version: 8.19 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: ntdll Assignee: wine-bugs(a)winehq.org Reporter: novae.harpist06(a)icloud.com Distribution: --- Created attachment 75493 --> https://bugs.winehq.org/attachment.cgi?id=75493 Reproducer NtQuerySystemInformation with SystemProcessInformation returns misaligned entries. Reproducer attached. Windows output: Process name: (null) Process name: System Process name: Secure System Process name: Registry ... (trimmed) Wine output: Process name: Z Misaligned at entry 1 -- 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=55928 Fabian Maurer <dark.shadow4(a)web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dark.shadow4(a)web.de -- 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=55928 --- Comment #1 from Fabian Maurer <dark.shadow4(a)web.de> --- Not sure how big the impact is, but yes that's a deviation. Do you know a program that relies on that alignment? -- 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=55928 --- Comment #2 from novae.harpist06(a)icloud.com --- Any program as long as you compile it with clang and ubsan. This is how I ran into the issue. -- 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=55928 Bernhard Übelacker <bernhardu(a)mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bernhardu(a)mailbox.org --- Comment #3 from Bernhard Übelacker <bernhardu(a)mailbox.org> --- I could reproduce it with compiling with this line: clang++ -O0 -g -Wl,/debug -fsanitize=undefined -lntdll main.cpp -o main.exe Following modification would fix the alignment. Nearby there are similar lines to align the "pos" pointer. dlls/ntdll/unix/system.c @@ -2527,2 +2527,3 @@ static unsigned int get_system_process_info + (name_len + 1) * sizeof(WCHAR); + proc_len = (proc_len + 7) & ~7; *len += proc_len; -- 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.
http://bugs.winehq.org/show_bug.cgi?id=55928 Bernhard Übelacker <bernhardu(a)mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #75493|0 |1 is obsolete| | --- Comment #4 from Bernhard Übelacker <bernhardu(a)mailbox.org> --- Created attachment 78550 --> http://bugs.winehq.org/attachment.cgi?id=78550 Reproducer, main.cpp, adjusted to actually print the ubsan output. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=55928 --- Comment #5 from Bernhard Übelacker <bernhardu(a)mailbox.org> --- Created attachment 78551 --> http://bugs.winehq.org/attachment.cgi?id=78551 Output from Reproducer -- 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.
http://bugs.winehq.org/show_bug.cgi?id=55928 --- Comment #6 from Bernhard Übelacker <bernhardu(a)mailbox.org> --- Submitted merge request: https://gitlab.winehq.org/wine/wine/-/merge_requests/8043 -- 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.
http://bugs.winehq.org/show_bug.cgi?id=55928 Bernhard Übelacker <bernhardu(a)mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Fixed by SHA1| |40bab0cfc6be65e43a565531119 | |347c60c886c6f Resolution|--- |FIXED --- Comment #7 from Bernhard Übelacker <bernhardu(a)mailbox.org> --- https://gitlab.winehq.org/wine/wine/-/commit/40bab0cfc6be65e43a565531119347c... Merge request got committed, resolving fixed. -- 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.
http://bugs.winehq.org/show_bug.cgi?id=55928 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #8 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 10.9. -- 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