- Wine-GitLab - list.winehq.org

[PATCH 0/1] MR8899: ntdll: Fix crash in check_invalid_gsbase() on macOS.
by Brendan Shanks (＠bshanks) Sept. 4, 2025

Sept. 4, 2025

I saw a crash when Unix code would try to execute a NULL function pointer: while handling the SEGV `check_invalid_gsbase()` would try to deference `%rip` which is of course NULL. This was only seen on macOS because the initial `if (cur_gsbase == (ULONG_PTR)teb)` check will never be equal, since there's no way [1] to find what GSBASE was when the signal was raised. macOS uses `%gs` for it's own thread-local storage, and we have to reset GSBASE to the Mac TSD in `init_handler()` before anything else runs. This means that every page fault will result in reading from `%rip`, and `virtual_uninterrupted_read_memory()` must be used in case `%rip` isn't a valid address. I also added some comments to explain the lack of a Mac codepath for getting/setting GSBASE. With this commit and testing with the sample code in #57444, I found that on Intel Macs `check_invalid_gsbase()` is necessary and works correctly. Rosetta does not implement resetting GSBASE on a `pop %gs` though, so `check_invalid_gsbase()` isn't necessary there. I do worry slightly about the overhead of calling `virtual_uninterrupted_read_memory()` for every page fault; if that ever needs to be reduced, skipping `check_invalid_gsbase()` on Rosetta would be a first step. [1]: This is mostly but not completely true--when a user LDT is installed (i.e. wow64), macOS uses a larger "full" sigcontext that includes the value of GSBASE. But Rosetta doesn't implement this, so it's only useful on Intel Macs. And if we wanted to use it on Intel Macs, we would need to always install a user LDT. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8899

2 1

Re: [PATCH v2 0/5] MR8882: Cmd engine rewrite (part XXXIV) - approved
by Alexandre Julliard (＠julliard) Sept. 3, 2025

Sept. 3, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8882

1 0

Re: [PATCH v2 0/6] MR8892: Comment or define undefined WinRT member interfaces (1/2) - approved
by Alexandre Julliard (＠julliard) Sept. 3, 2025

Sept. 3, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8892

1 0

Re: [PATCH v2 0/1] MR8893: opcservices: Remove separate part name allocation. - approved
by Alexandre Julliard (＠julliard) Sept. 3, 2025

Sept. 3, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8893

1 0

Re: [PATCH v2 0/2] MR8897: gitlab: Merge the platform docker files back into a single file. - approved
by Alexandre Julliard (＠julliard) Sept. 3, 2025

Sept. 3, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8897

1 0

Re: [PATCH v2 0/2] MR8873: Iphlpapi, nsiproxy.sys: Fix async icmp echo request completion. - approved
by Alexandre Julliard (＠julliard) Sept. 3, 2025

Sept. 3, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8873

1 0

Re: [PATCH v2 0/1] MR8860: winex11: Bitmap rendering now checks pbuffer bit instead of pixmap bit - approved
by Alexandre Julliard (＠julliard) Sept. 3, 2025

Sept. 3, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8860

1 0

Re: [PATCH v17 0/5] MR8703: combase: Implement HSTRING marshaling functions. - approved
by Alexandre Julliard (＠julliard) Sept. 3, 2025

Sept. 3, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8703

1 0

Re: [PATCH v2 0/1] MR8854: combase/tests: Suppress trace messages from the test dll. - approved
by Alexandre Julliard (＠julliard) Sept. 3, 2025

Sept. 3, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8854

1 0

Re: [PATCH v2 0/2] MR8881: mshtml: Case insensitive lookup and element prototype fixes. - approved
by Alexandre Julliard (＠julliard) Sept. 3, 2025

Sept. 3, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8881

1 0