- Wine-GitLab - list.winehq.org

Re: [PATCH v3 0/2] MR8894: crypt32: Accept PKCS12_ALWAYS_CNG_KSP flag and fall back to standard import.
by Christian Tinauer (＠christiantinauer) Sept. 4, 2025

Sept. 4, 2025

On Wed Sep 3 15:12:52 2025 +0000, Christian Tinauer wrote: > changed this line in [version 2 of the diff](/wine/wine/-/merge_requests/8894/diffs?diff_id=205547&start_sha=a38d27fbc01002ca53b5c64949790f1a0a326477#85309c8ce7c9487572d2f5c6bde361a8bb004951_3315_3315) Done. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8894#note_114847

1 0

Re: [PATCH v3 0/2] MR8894: crypt32: Accept PKCS12_ALWAYS_CNG_KSP flag and fall back to standard import.
by Christian Tinauer (＠christiantinauer) Sept. 4, 2025

Sept. 4, 2025

On Thu Sep 4 08:50:16 2025 +0000, Hans Leidekker wrote: > That should be a FIXME. Done. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8894#note_114846

1 0

Re: [PATCH v10 0/2] MR7064: ntdll: Check for invalid GS base in the 64-bit segv_handler.
by Brendan Shanks (＠bshanks) Sept. 4, 2025

Sept. 4, 2025

I created !8899 to fix a crash on macOS in `check_invalid_gsbase()`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7064#note_114829

1 0

[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

Re: [PATCH v2 0/1] MR8884: quartz/tests: Fix a failing test in test_video_window_owner(). - 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/8884

1 0

Re: [PATCH v2 0/1] MR8885: winevulkan: Handle NULL buffer pointer in vkFreeCommandBuffers(). - 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/8885

1 0

Re: [PATCH v2 0/6] MR8889: win32u: Move device memory object wrapper from winevulkan. - 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/8889

1 0

Re: [PATCH v2 0/1] MR8896: secur32: Avoid syscall fault in process_detach() if GnuTLS failed to load. - 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/8896

1 0

Re: [PATCH v2 0/1] MR8886: crypt32: Avoid syscall fault in process_detach() if GnuTLS failed to load. - 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/8886

1 0

Re: [PATCH v6 0/1] MR8888: opengl32/tests: Add 16-bit bitmap rendering tests - 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/8888

1 0

[PATCH 0/1] MR8898: win32u: Bump AMD internal driver version.
by Paul Gofman (＠gofman) Sept. 3, 2025

Sept. 3, 2025

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/8898

2 1

[PATCH v2 0/1] MR8858: sane.ds: Adding saving scanner settings to the registry
by Ivan Lyugaev (＠valy) Sept. 3, 2025

Sept. 3, 2025

The latest scanner settings are now automatically saved in the HKEY_CURRENT_USER\Software\ScannersSettings\Vendor\Product path. This modification was tested and worked correctly. -- v2: sane.ds: Adding saving scanner settings to the registry https://gitlab.winehq.org/wine/wine/-/merge_requests/8858

3 4

[PATCH v2 0/4] MR8895: Draft: wintypes: Implement a generic thread-safe IMap<HSTRING, IInspectable>.
by Vibhav Pant (＠vibhavp) Sept. 3, 2025

Sept. 3, 2025

-- v2: wintypes: Implement a generic IMap<HSTRING, IInspectable>. https://gitlab.winehq.org/wine/wine/-/merge_requests/8895

3 5

[PATCH 0/4] MR8851: tools: Install missing 64bit gstreamer runtime / -dev packages.
by Rémi Bernon Sept. 3, 2025

Sept. 3, 2025

It's not normally possible to cross install gstreamer -dev packages on Debian, because of some conflicting gir files. We don't need these and we only care about headers and libraries. Extracting the packages contents manually instead works just fine. This is what I've been doing for my test VMs images. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8851

3 13

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

Sept. 3, 2025

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/8897

2 2

[PATCH 0/1] MR8896: secur32: Avoid syscall fault in process_detach() if GnuTLS failed to load.
by Brendan Shanks (＠bshanks) Sept. 3, 2025

Sept. 3, 2025

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/8896

3 2