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

[PATCH 0/2] MR8632: dsound: Zero IDirectSoundBufferImpl_Lock out params if the arguments are invalid.
by Alfred Agrell (＠Alcaro) 31 Jul '25

31 Jul '25

https://store.steampowered.com/app/638930/Dal_Segno/ occasionally calls IDirectSoundBuffer::Lock() with writecursor == buflen and writebytes == 0. This is illegal, so Lock() returns DSERR_INVALIDPARAM - but the game doesn't notice, and does a memcpy to fill the returned buffer. On native, the out params are zeroed on failure, so the memcpy ends up doing the right thing. On Wine, the out params are left unchanged on failure, resulting in memcpy(STACK GARBAGE, src, STACK GARBAGE); after a few minutes of gameplay or idling (when the size isn't coincidentally zero), it crashes. ``` 6518.932:0130:01b8:trace:dsound:IDirectSoundBufferImpl_GetCurrentPosition (01FF9BC0,090BEE94,00000000) 6518.932:0130:01b8:trace:dsound:IDirectSoundBufferImpl_GetCurrentPosition playpos = 20674080, writepos = -1, buflen=20709376 (01FF9BC0, time=6518932) 6518.932:0130:01b8:trace:dsound:IDirectSoundBufferImpl_Lock (01FF9BC0,20709376,0,090BEEA0,090BEE90,090BEE9C,090BEE98,0x00000000) at 6518932 6518.932:0130:01b8:warn:dsound:IDirectSoundBufferImpl_Lock Invalid parameter, writecursor: 20709376 >= buflen: 20709376 6518.932:0130:01b8:trace:seh:dispatch_exception code=c0000005 flags=0 addr=00444C5C ip=00444c5c 6518.932:0130:01b8:trace:seh:dispatch_exception info[0]=00000001 6518.932:0130:01b8:trace:seh:dispatch_exception info[1]=0042d0b5 6518.932:0130:01b8:warn:seh:dispatch_exception EXCEPTION_ACCESS_VIOLATION exception (code=c0000005) raised 6518.932:0130:01b8:trace:seh:dispatch_exception eax=0000008e ebx=7aa539c0 ecx=00000008 edx=00000000 esi=004dcf10 edi=0042d0b5 6518.932:0130:01b8:trace:seh:dispatch_exception ebp=090bee68 esp=090bee60 cs=0023 ss=002b ds=002b es=002b fs=0063 gs=006b flags=00010246 ``` (The game is poorly written in several other ways - the segfault is caught in a SEH handler that also segfaults, repeat until stack overflow; it repeatedly unlocks critical section 004A4A28, which nothing ever locks; there's a homemade spinlock implementation at 0042CAB2; probably some more weirdness I didn't discover. But those have little or no user-visible impact, they just look ugly in the PROTON_LOG and/or game disassembly. Let's ignore them.) ...is there such a thing as too many tests? Nobody's gonna try most of what these tests do. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8632

3 4

[PATCH v2 0/5] MR8681: widl: Require runtimeclass contract or version attribute.
by Hans Leidekker (＠hans) 31 Jul '25

31 Jul '25

-- v2: widl: Require runtimeclass contract or version attribute. include: Add missing runtimeclass contract attributes. widl: Check composition method parameters. widl: Check activation method parameters. include: Comment reference to undefined activation interface. https://gitlab.winehq.org/wine/wine/-/merge_requests/8681

3 8

[PATCH 0/1] MR8682: winebus.sys: Prefer hidraw for all Virpil (VID 3344) devices.
by Rémi Bernon 31 Jul '25

31 Jul '25

Original patch from Ty (Chaos). Re-authored from https://gitlab.winehq.org/wine/wine/-/merge_requests/8667, the change doesn't seem to require accurate authorship and the user prefers not to have their full name mentioned. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8682

1 2

[PATCH 0/5] MR8681: widl: Require runtimeclass contract or version attribute.
by Hans Leidekker (＠hans) 31 Jul '25

31 Jul '25

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

3 7

[PATCH 0/1] MR8680: winebus.sys: Fix DualSense BT quirk.
by Arek Hiler (＠ivyl) 31 Jul '25

31 Jul '25

We need to memmove 11 bytes, otherwise `buffer[10]` ends up as a copy of `buffer[9]` which resulted in the second byte of button state being wrongly re-used as a third byte as well. The original code did not suffer from this - while the length check was also one off, and likely informed the one off memmove, the buffer pointer was just incremented so everything "moved" correctly. Fixes: c708295ed6de ("winebus: Move Sony controllers report fixups to PE side.")<br/> Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56883<br/> Signed-off-by: Arkadiusz Hiler \<ahiler(a)codeweavers.com\></br> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8680

3 6

Re: [PATCH v2 0/1] MR8660: msado15: Implement ADOConnectionConstruction15::get_DSO
by Rémi Bernon 31 Jul '25

31 Jul '25

This broke the tests on Windows. https://test.winehq.org/data/patterns.html#msado15:msado15 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8660#note_111610

1 0

[PATCH v5 0/9] MR8664: cryptxml: Implement some stubs.
by Mohamad Al-Jaf (＠maljaf) 31 Jul '25

31 Jul '25

Needed by Lenovo USBRecoveryCreator. -- v5: cryptxml/tests: Add some signature verification tests. cryptxml: Implement CryptXmlGetStatus(). cryptxml: Implement CryptXmlVerifySignature() stub. cryptxml: Partially implement CryptXmlGetSignature(). cryptxml: Implement CryptXmlGetDocContext(). cryptxml: Implement CryptXmlClose(). cryptxml: Implement CryptXmlOpenToDecode() stub. https://gitlab.winehq.org/wine/wine/-/merge_requests/8664

3 14

[PATCH v45 0/6] MR6766: wintypes: Add stubs for Windows.Foundation.Collections.PropertySet.
by Vibhav Pant (＠vibhavp) 31 Jul '25

31 Jul '25

-- v45: wintypes: Add stubs for IIterable<IKeyValuePair<HSTRING, IInspectable *>> to PropertySet. wintypes: Add stubs for IMap<HSTRING, IInspectable *> to PropertySet. wintypes: Add stubs for IObservableMap<HSTRING, IInspectable *> to PropertySet. wintypes: Add stubs for Windows.Foundation.Collections.PropertySet. https://gitlab.winehq.org/wine/wine/-/merge_requests/6766

3 8

[PATCH v2 0/2] MR8662: Revert "mmdevapi: Error out if the channel count or sampling rate doesn't match the mix format."
by Giovanni Mascellani (＠giomasce) 31 Jul '25

31 Jul '25

This reverts commit 00211db0d08d60ee9a0e40206bf7cf9b5b88987b, which introduced a lot of test failures and is taking some time to be reworked properly. -- v2: mmdevapi/tests: Mark a wrong error code by IsFormatSupported() as todo. https://gitlab.winehq.org/wine/wine/-/merge_requests/8662

3 3

[PATCH 0/3] MR8674: win32u: Update DC OpenGL drawable when it is acquired.
by Rémi Bernon 31 Jul '25

31 Jul '25

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58448 Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58459 Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58488 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8674

2 4