June 2025 - Wine-gitlab - list.winehq.org

[PATCH v2 0/1] MR8205: comctl32: Make sure line_widths is big enough.
by Yuxuan Shui (＠yshui) 06 Jun '25

06 Jun '25

When button_count is 0, we would allocate 0 bytes for line_widths, but later line_count would be 1 so we would still try to access line_widths[0] which is out-of-bound. -- v2: comctl32: Make sure line_widths is big enough. https://gitlab.winehq.org/wine/wine/-/merge_requests/8205

4 6

[PATCH 0/5] MR8222: widl: Add rows for the struct type.
by Hans Leidekker (＠hans) 06 Jun '25

06 Jun '25

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

3 7

[PATCH v5 0/5] MR8206: windows.devices.bluetooth: Add stubs for (I)BluetoothDeviceStatics.
by Vibhav Pant (＠vibhavp) 06 Jun '25

06 Jun '25

-- v5: windows.devices.bluetooth: Add stub for IBluetoothDeviceStatics. windows.devices.bluetooth/tests: Add tests for BluetoothDeviceStatics. include: Add definitions for Windows.Devices.Bluetooth.BluetoothDevice. include: Add windows.devices.bluetooth.rfcomm.idl. https://gitlab.winehq.org/wine/wine/-/merge_requests/8206

3 14

[PATCH 0/1] MR8221: urlmon: Skip fragment part when checking filenames for file protocol.
by Gabriel Ivăncescu 06 Jun '25

06 Jun '25

I have no idea what the previous test was trying to do, but it was clearly wrong and misleading. First, it placed the fragment for no reason in file_name_buf because it then replaced it with a NUL, making you believe it's part of it but when it really was not. Then, it opened the original index.html to write XXX to it, so the fragment filename never actually existed in the first place, and was never tested. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8221

2 2

[PATCH 0/2] MR8196: winewayland: Ignore text-input "done" events that don't modify state.
by Alexandros Frantzis (＠afrantzis) 06 Jun '25

06 Jun '25

Compositors send a "done" event after every text-input commit, even if the reported state (preedit etc) hasn't changed. Acting on such events is at best wasteful, but can additionally lead to incorrect IME related effects (e.g., deleting the currently selected text), so ignore them, similarly to what Qt and GTK do. The first commit performs some state related cleanups. -- This solves an IME problem I have been experiencing with compositors that support text-input-v3 (e.g., kwin). 1. Run notepad 2. Write some text 3. Try to select text Expected result: text is selected Actual result: text is deleted What happens is that while the user is selecting text the driver gets a series of `SetIMECompositionRect` callback which cause a series of text-input-v3 `set_cursor_rectangle`-`commit` requests. These `commit`s elicit `done` events from the compositor with no state changes, which we interpret as "clear the composition string" (and by extension the active selection), because that's what such states normally means, and forward them to windows IME. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8196

4 4

[PATCH v2 0/4] MR8220: comdlg32: Don't use a shadow devmode structure in PrintDlg.
by Jacob Czekalla (＠JacobCzekalla) 06 Jun '25

06 Jun '25

If the "Properties" button is clicked in the printdlg it opens the print properties dlg. The problem is the values set in the properties dlg don't get passed through correctly to the final printing stage. This prevents properties like duplex and page size from working. -- v2: comdlg32: Don't use a shadow devmode structure in PrintDlgA. comdlg32/tests: Add tests for changing devmode properties in the hook procedure for PrintDlgA. comdlg32: Don't use a shadow devmode structure in PrintDlgW. comdlg32/tests: Add tests for changing devmode properties in the hook procedure for PrintDlgW. https://gitlab.winehq.org/wine/wine/-/merge_requests/8220

3 5

[PATCH v3 0/1] MR8145: ldap: Avoid code duplication between sasl_client_start() and sasl_client_step().
by Dmitry Timoshkov (＠dmitry) 06 Jun '25

06 Jun '25

-- v3: ldap: Avoid code duplication between sasl_client_start() and sasl_client_step(). https://gitlab.winehq.org/wine/wine/-/merge_requests/8145

3 2

[PATCH v2 0/1] MR8217: wine.inf: add mandatory Parameters for Tcpip6 and Dnscache services
by Vitaly Lipatov (＠lav) 06 Jun '25

06 Jun '25

Add subkey Tcpip6/Parameters and Dnscache/Parameters. They are required by some progs. -- v2: wine.inf: add subkey Parameters for Tcpip6 and Dnscache services https://gitlab.winehq.org/wine/wine/-/merge_requests/8217

3 2

[PATCH v2 0/2] MR8216: server: prevent division by zero in coordinate mapping
by Wei Xie (＠wayne) 05 Jun '25

05 Jun '25

Wine applications crash in VNC/headless environments due to invalid all-zero monitor coordinates. The root cause is: - Environment Issue: VNC systems report all displays as RR_Disconnected - Faulty Fallback: Wine incorrectly reverts to legacy XRandR 1.0 API when no "connected" displays are detected - API Incompatibility: Modern XRandR 1.6 doesn't support Wine's XRandR 1.0 implementation, causing XRRSizes() to return empty display modes - Uninitialized Data: This leaves critical display fields (dmPelsWidth/Height) at initialization value 0 -- v2: winex11: handle fallback display modes when XRandR fails server: prevent division by zero in coordinate mapping https://gitlab.winehq.org/wine/wine/-/merge_requests/8216

2 2

[PATCH 0/1] MR8216: server: prevent division by zero in coordinate mapping
by Wei Xie (＠wayne) 05 Jun '25

05 Jun '25

Wine applications crash in VNC/headless environments due to invalid all-zero monitor coordinates. The root cause is: Environment Issue: VNC systems report all displays as RR_Disconnected Faulty Fallback: Wine incorrectly reverts to legacy XRandR 1.0 API when no "connected" displays are detected API Incompatibility: Modern XRandR 1.6 doesn't support Wine's XRandR 1.0 implementation, causing XRRSizes() to return empty display modes Uninitialized Data: This leaves critical display fields (dmPelsWidth/Height) at initialization value 0 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8216

2 2