March 2026 - Wine-gitlab - list.winehq.org

[PATCH 0/1] MR10244: Draft: vbscript: fix sub first arg parentheses handling
by Francis De Brabandere (＠francisdb) 04 Mar '26

04 Mar '26

Closes https://bugs.winehq.org/show_bug.cgi?id=54177 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10244

2 1

[PATCH v5 0/2] MR9514: gdi32: Fix font creation with invaild bits.
by Bernhard Kölbl (＠besentv) 04 Mar '26

04 Mar '26

Black Squad's launcher creates a font like ```c font = CreateFontW(12, 0, 0, 0, 0, 0, 0, 0, DEFAULT_CHARSET, 0, 0, 0, 0x80 /* ??? */ | FF_ROMAN, L""); ``` This confuses Wine and causes it to pick a sans serif font instead of a serif one. -- v5: win32u: Mask highest font family bit to zero. https://gitlab.winehq.org/wine/wine/-/merge_requests/9514

2 3

Re: [PATCH v4 0/2] MR9514: gdi32: Fix font creation with invaild bits.
by Bernhard Kölbl (＠besentv) 04 Mar '26

04 Mar '26

On Wed Mar 4 13:20:45 2026 +0000, Bernhard Kölbl wrote: > changed this line in [version 4 of the diff](/wine/wine/-/merge_requests/9514/diffs?diff_id=248410&start_sha=9fc983e1c145e2bd5093842c80df185a283ac390#9a562c66de5ac819e19590d9210cef57a29e7f04_7962_8075) I haven't seen that happening. Changed it to 0xf0. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9514#note_131260

1 0

Re: [PATCH v3 0/2] MR9514: gdi32: Fix font creation with invaild bits.
by Bernhard Kölbl (＠besentv) 04 Mar '26

04 Mar '26

On Sat Feb 7 10:45:25 2026 +0000, Huw Davies wrote: > What's going on here? Why is this always failing with Wine? Actually I'm not entirely sure what's going on here, the test results from my Linux machine are: ```log mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Sans" for height 6, expected L"Small Fonts" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Sans" for height 12, expected L"Small Fonts" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Sans" for height 16, expected L"MS Sans Serif" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Sans" for height 24, expected L"MS Sans Serif" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Sans" for height 32, expected L"Arial" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Sans" for height 96, expected L"Arial" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Serif" for height 6, expected L"Times New Roman" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Serif" for height 12, expected L"Times New Roman" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Serif" for height 16, expected L"MS Serif" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Serif" for height 24, expected L"Times New Roman" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Serif" for height 32, expected L"Times New Roman" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Serif" for height 96, expected L"Times New Roman" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Serif" for height 6, expected L"Times New Roman" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Sans" for height 12, expected L"Small Fonts" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Serif" for height 16, expected L"MS Serif" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Serif" for height 24, expected L"Times New Roman" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Sans" for height 32, expected L"Arial" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Mono" for height 48, expected L"Courier New" mfont.c:8081: Test marked todo: Got unexpected font L"Liberation Sans" for height 96, expected L"Arial" ``` I don't think we have Arial in upstream, however the other fonts should definitely be there. Want me to further investigate this? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/9514#note_131255

1 0

[PATCH v3 0/3] MR10114: msi: Handle unusual installer files (padding, malformed strrefs)
by Theodoros Chatzigiannakis (＠TChatzigiannakis) 04 Mar '26

04 Mar '26

This PR addresses two issues that I observed preventing the Native Instruments Kontakt Player MSI (and other MSI files from the same vendor) from successfully finishing: 1. Some installers contain padding for 4-byte alignment of table streams. * Proposed fix: We relax Wine's parsing to allow for such padding. 2. Some installers declare that they have 3-byte strrefs, but the data actually contains 2-byte strrefs. This looks like a malformed MSI to me, and Wine correctly rejects them currently. The regression tests in this PR indicate that Windows in CI (test-win10-21h2-\*) is reading garbage as well. However, NI ships these MSI files, which implies that somehow the installation successfully completes on Windows machines nevertheless. * Proposed fix: We explicitly detect and correct this particular mismatch at load time of the MSI, in order to be able to complete the installation under Wine as well. --- The above are technically separate issues, but they affect the same software in similar ways, so I opted for one PR. Please me know if I should split them. --- Quick repro hint: Native Instruments offers a program called Native Access (which acts as an installation hub on the user's machine). On user request, NA downloads and launches the affected MSI files in the background. If you try to repro, a couple of things to watch out for: 1. NA seems to always report a successful installation, even when the underlying MSI fails to complete (and therefore nothing is installed). This PR does nothing to address this false positive, as it is most likely a logic bug in the hub or installation orchestration. 2. To observe/examine the underlying MSI files (including the fix presented here), you'll need to grab them after NA downloads and extracts them on your machine (or find manual/direct downloads on the NI website, where applicable) --- _Disclaimers for reviewers:_ * _AI was used while tracking down the issues and coming up with fixes._ * _I am not affiliated with Native Instruments, I'm just a user._ -- v3: msi/tests: Add regression tests for table stream padding and mismatched string refs. https://gitlab.winehq.org/wine/wine/-/merge_requests/10114

2 4

Re: [PATCH v4 0/3] MR10175: winmm/tests: Test a curated list of wave formats.
by Giovanni Mascellani (＠giomasce) 04 Mar '26

04 Mar '26

On Wed Mar 4 11:22:01 2026 +0000, Rémi Bernon wrote: > Fwiw the winmm:wave tests are failing on several Windows and Linux VMs (https://test.winehq.org/data/patterns.html#winmm:wave) Thanks, I'll have a look. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10175#note_131251

1 0

Re: [PATCH v5 0/2] MR10067: wineandroid: fix Android NDK/ARM build issues (resubmission with expanded rationale)
by Twaik Yont (＠twaik) 04 Mar '26

04 Mar '26

@julliard can you please review !9874 as well please? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10067#note_131248

1 0

[PATCH v12 0/3] MR9874: wineandroid.drv: experimental bring-up fixes for Android 7.1.x
by Twaik Yont (＠twaik) 04 Mar '26

04 Mar '26

Requires !10067, fixup/followup to e0d3683d8947cd9a3cb20acdf69091b183a90861 and 4b5311c7e02aa7ab2409f8fdcd22233f116dc4bc. #### Note on authorship and language This text was translated into English and structured with the assistance of ChatGPT. The author is not a native English speaker and may struggle with written technical English; any wording issues are unintentional. #### Description This merge request is an experimental attempt to bring wineandroid.drv back to a minimally functional state on Android 7.1.x. The current Android support appears to have bit-rotted over time, likely due to a combination of internal contract changes in Wine, toolchain and NDK evolution, and the absence of continuous Android testing. The changes proposed here aim to re-establish a working baseline and make the regressions more concrete and easier to reason about. #### Test environment and context All testing so far was performed on Android 7.1.2 x86 images from the Android-x86 project, published on osboxes.org ([link](https://www.osboxes.org/android-x86/)) The primary target is i386 Wine running on x86 Android userspace. Because no up-to-date official Android build scripts were available, Wine was built using Termux as an unsupported but convenient environment for rapid testing. The Java part (WineActivity and related glue code) was built with `sharedUserId = com.termux` purely for convenience (filesystem access and faster prototyping). **Termux is explicitly not considered a supported environment and is mentioned only to provide context for the testing setup.** The build uses Android NDK r29. Since the last known functional Android Wine builds, the NDK and linker behavior have changed multiple times, including symbol visibility and relocation handling. These changes appear to have directly affected several failure modes described below. There is also a known issue with wine-preloader producing excessively large binaries under recent toolchains; a workaround exists but is not included here and is mentioned only as additional context. #### Scope of changes This merge request currently modifies only Android-specific code: - `wineandroid.drv` (native code and Java glue, including WineActivity) - android-only paths in `dlls/ntdll/unix/loader.c` No non-Android platforms are affected. #### Summary of resolved issues 1) Desktop initialization ordering regression After recent `CreateDesktopW`-related changes, `ANDROID_CreateDesktop` started running before the DESKTOP window was created. The Java glue assumed the opposite order. The fix splits desktop initialization into two phases (view setup first, desktop window attachment later via an explicit is_desktop flag) and corrects the `ANDROID_CreateDesktop` success return value. 2) Rendering stalls and `USER lock assertion` Occasional crashes with: ``` err:system:user_check_not_lock BUG: holding USER lock ``` A full memory barrier was added in the dequeue path to avoid a likely memory ordering issue. Additionally, a guard was added for a `NULL` buffer returned on successful dequeue. 3) Early startup failure in `virtual_alloc_first_teb` Wine frequently failed with: ``` err:virtual:virtual_alloc_first_teb wine: failed to map the shared user data: c0000018 ``` `virtual_init()` is now called from `JNI_OnLoad()` to reduce the likelihood of the JVM reserving the required address range before Wine initializes its virtual memory subsystem. #### Remaining issues - Window focus handling is partially broken: windows receive focus on click, but are not always raised to the front. - Despite the available desktop area being larger (e.g. 1024x696), Wine consistently reports a display resolution of 800x600. #### Limitations Wine on Android stopped functioning entirely starting with Android 8. As a result, all testing so far has been limited to Android 7.1.x. No claims are made about behavior on newer Android versions. #### Out of scope This merge request intentionally does not attempt to: - redesign IPC mechanisms - introduce new Android integration models - address Android 8+ compatibility The sole goal is to re-establish a minimally working baseline on Android 7.1.x and enable informed discussion about what regressed and how to proceed. **This is my first contribution to Wine, and I am aware that some parts of this analysis or implementation may be flawed. I would be grateful for any review comments or suggestions.** <details> <summary>Also, the screenshot of the working prototype.</summary> ![1](/uploads/666d78a712edce531a25521a2615ea9f/1.png){width=756 height=600} </details> -- v12: ntdll: call virtual_init from JNI_OnLoad to avoid early virtual_alloc_first_teb failure Wineandroid: Sanitize dequeueBuffer result and reject missing buffers. wineandroid: fix desktop init ordering after CreateDesktopW changes https://gitlab.winehq.org/wine/wine/-/merge_requests/9874

3 4

[PATCH v11 0/3] MR9874: wineandroid.drv: experimental bring-up fixes for Android 7.1.x
by Twaik Yont (＠twaik) 04 Mar '26

04 Mar '26

Requires !10067, fixup/followup to e0d3683d8947cd9a3cb20acdf69091b183a90861 and 4b5311c7e02aa7ab2409f8fdcd22233f116dc4bc. #### Note on authorship and language This text was translated into English and structured with the assistance of ChatGPT. The author is not a native English speaker and may struggle with written technical English; any wording issues are unintentional. #### Description This merge request is an experimental attempt to bring wineandroid.drv back to a minimally functional state on Android 7.1.x. The current Android support appears to have bit-rotted over time, likely due to a combination of internal contract changes in Wine, toolchain and NDK evolution, and the absence of continuous Android testing. The changes proposed here aim to re-establish a working baseline and make the regressions more concrete and easier to reason about. #### Test environment and context All testing so far was performed on Android 7.1.2 x86 images from the Android-x86 project, published on osboxes.org ([link](https://www.osboxes.org/android-x86/)) The primary target is i386 Wine running on x86 Android userspace. Because no up-to-date official Android build scripts were available, Wine was built using Termux as an unsupported but convenient environment for rapid testing. The Java part (WineActivity and related glue code) was built with `sharedUserId = com.termux` purely for convenience (filesystem access and faster prototyping). **Termux is explicitly not considered a supported environment and is mentioned only to provide context for the testing setup.** The build uses Android NDK r29. Since the last known functional Android Wine builds, the NDK and linker behavior have changed multiple times, including symbol visibility and relocation handling. These changes appear to have directly affected several failure modes described below. There is also a known issue with wine-preloader producing excessively large binaries under recent toolchains; a workaround exists but is not included here and is mentioned only as additional context. #### Scope of changes This merge request currently modifies only Android-specific code: - `wineandroid.drv` (native code and Java glue, including WineActivity) - android-only paths in `dlls/ntdll/unix/loader.c` No non-Android platforms are affected. #### Summary of resolved issues 1) Desktop initialization ordering regression After recent `CreateDesktopW`-related changes, `ANDROID_CreateDesktop` started running before the DESKTOP window was created. The Java glue assumed the opposite order. The fix splits desktop initialization into two phases (view setup first, desktop window attachment later via an explicit is_desktop flag) and corrects the `ANDROID_CreateDesktop` success return value. 2) Rendering stalls and `USER lock assertion` Occasional crashes with: ``` err:system:user_check_not_lock BUG: holding USER lock ``` A full memory barrier was added in the dequeue path to avoid a likely memory ordering issue. Additionally, a guard was added for a `NULL` buffer returned on successful dequeue. 3) Early startup failure in `virtual_alloc_first_teb` Wine frequently failed with: ``` err:virtual:virtual_alloc_first_teb wine: failed to map the shared user data: c0000018 ``` `virtual_init()` is now called from `JNI_OnLoad()` to reduce the likelihood of the JVM reserving the required address range before Wine initializes its virtual memory subsystem. #### Remaining issues - Window focus handling is partially broken: windows receive focus on click, but are not always raised to the front. - Despite the available desktop area being larger (e.g. 1024x696), Wine consistently reports a display resolution of 800x600. #### Limitations Wine on Android stopped functioning entirely starting with Android 8. As a result, all testing so far has been limited to Android 7.1.x. No claims are made about behavior on newer Android versions. #### Out of scope This merge request intentionally does not attempt to: - redesign IPC mechanisms - introduce new Android integration models - address Android 8+ compatibility The sole goal is to re-establish a minimally working baseline on Android 7.1.x and enable informed discussion about what regressed and how to proceed. **This is my first contribution to Wine, and I am aware that some parts of this analysis or implementation may be flawed. I would be grateful for any review comments or suggestions.** <details> <summary>Also, the screenshot of the working prototype.</summary> ![1](/uploads/666d78a712edce531a25521a2615ea9f/1.png){width=756 height=600} </details> -- v11: ntdll: call virtual_init from JNI_OnLoad to avoid early virtual_alloc_first_teb failure Wineandroid: Sanitize dequeueBuffer result and reject missing buffers. wineandroid: fix desktop init ordering after CreateDesktopW changes https://gitlab.winehq.org/wine/wine/-/merge_requests/9874

3 8

[PATCH 0/5] MR10243: mmdevapi: Take the wave format validation code from the tests.
by Giovanni Mascellani (＠giomasce) 04 Mar '26

04 Mar '26

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

2 5