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

[PATCH v7 0/2] MR8671: cryptowinrt: Stub ICryptographicBufferStatics::EncodeToBase64String().
by Mohamad Al-Jaf (＠maljaf) 03 Aug '25

03 Aug '25

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55759 -- v7: cryptowinrt/tests: Add some ICryptographicBufferStatics::EncodeToBase64String() tests. cryptowinrt: Stub ICryptographicBufferStatics::EncodeToBase64String(). https://gitlab.winehq.org/wine/wine/-/merge_requests/8671

3 4

[PATCH v3 0/9] MR8641: wined3d: Fixes for D3DTSS_TEXTURETRANSFORMFLAGS.
by Elizabeth Figura (＠zfigura) 01 Aug '25

01 Aug '25

-- v3: wined3d: Always divide 1.x projected textures by W in the FFP. wined3d: Always divide 1.x projected textures by W for shaders. wined3d: Initialize all remaining FFP texture coordinates to zero. wined3d: Copy the projective divisor in the FFP vertex pipeline. wined3d: Alter the texture matrix even for non-projected textures. wined3d: Pass 3 as the attribute count for generated texcoords. wined3d: Pass the attribute coordinate count to get_texture_matrix(). wined3d: Handle all invalid values in compute_texture_matrix(). d3d9/tests: Add comprehensive D3DTSS_TEXTURETRANSFORMFLAGS tests. https://gitlab.winehq.org/wine/wine/-/merge_requests/8641

3 10

[PATCH 0/1] MR8687: wined3d: Add GPU information for AMD NAVI44.
by Henri Verbeet (＠hverbeet) 01 Aug '25

01 Aug '25

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

3 2

[PATCH 0/1] MR8666: Revert "wined3d/glsl: Transpose the bump environment matrix."
by Elizabeth Figura (＠zfigura) 01 Aug '25

01 Aug '25

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

3 3

[PATCH v7 0/8] MR7797: ntdll: Wait for all pending operations to be cancelled in NtCancelIoFile[Ex]().
by Matteo Bruni (＠Mystral) 01 Aug '25

01 Aug '25

This fixes Trials Fusion often crashing when disconnecting a controller while there are more still connected. -- v7: hidclass: Set Status for pending IRPs of removed devices to STATUS_DEVICE_NOT_CONNECTED. ntdll/tests: Test IOSB values of the cancel operation. ntdll/tests: Add more NtCancelIoFile[Ex]() tests. ntdll: Wait for all asyncs to handle cancel in NtCancelIoFile[Ex](). server: Introduce a separate server object for the async cancel operation. server: Keep cancelled async in a separate list. server: Introduce a find_async_from_user helper. ntdll: Factor out a cancel_io() function. https://gitlab.winehq.org/wine/wine/-/merge_requests/7797

3 16

[PATCH v2 0/1] MR8639: rpcrt4: Don't read past the end of params in client_do_args.
by Yuxuan Shui (＠yshui) 01 Aug '25

01 Aug '25

* * * An example is `IDispatchEx_RemoteInvokeEx_Proxy` asan complaint: ``` 015c:0160:err:asan:asan_report ASan: read of 4 bytes at 015A88DC, caller 6A9A0EFA (__asan_report_load4_noabort, /home/shui/programs/wine/dlls/asan_dynamic_thunk/thunk.c:1031,1) 015c:0160:err:asan:asan_report stacktrace: 015c:0160:err:asan:asan_report 6A9A0EFA (__asan_report_load4_noabort, /home/shui/programs/wine/dlls/asan_dynamic_thunk/thunk.c:1031,1) 015c:0160:err:asan:asan_report 6A9ACB2C (client_do_args, /home/shui/programs/wine/dlls/rpcrt4/ndr_stubless.c:511,17) 015c:0160:err:asan:asan_report 6A9C5913 (ndr_client_call, /home/shui/programs/wine/dlls/rpcrt4/ndr_stubless.c:794,9) 015c:0160:err:asan:asan_report 6A965438 (NdrpClientCall2, /home/shui/programs/wine/dlls/rpcrt4/ndr_stubless.c:929,22) 015c:0160:err:asan:asan_report 6A916C77 (NdrClientCall2) 015c:0160:err:asan:asan_report 68662856 (IDispatchEx_RemoteInvokeEx_Proxy, dlls/dispex/i386-windows/disp_ex_p.c:82,15) 015c:0160:err:asan:asan_report 68661D6E (IDispatchEx_InvokeEx_Proxy, /home/shui/programs/wine/dlls/dispex/usrmarshal.c:100,10) 015c:0160:err:asan:asan_report 00401187 (IDispatchEx_InvokeEx, include/dispex.h:319,12) 015c:0160:err:asan:asan_report 00408F36 (test_dispex, /home/shui/programs/wine/dlls/dispex/tests/marshal.c:391,10) 015c:0160:err:asan:asan_report 004061FF (func_marshal, /home/shui/programs/wine/dlls/dispex/tests/marshal.c:433,5) 015c:0160:err:asan:asan_report 00407B60 (run_test, /home/shui/programs/wine/include/wine/test.h:794,9) 015c:0160:err:asan:asan_report 00407426 (main, /home/shui/programs/wine/include/wine/test.h:912,1) 015c:0160:err:asan:asan_report 0040767C (mainCRTStartup, /home/shui/programs/wine/dlls/msvcrt/crt_main.c:60,11) 015c:0160:err:asan:asan_report 7901FA24 (BaseThreadInitThunk) 015c:0160:err:asan:asan_report 7A50E0F7 (call_thread_func_wrapper) 015c:0160:err:asan:asan_report 7A5B6ADD (call_thread_func, /home/shui/programs/wine/dlls/ntdll/signal_i386.c:505,5) 015c:0160:err:asan:asan_report info: 015c:0160:err:asan:asan_report partial granule: 4 015c:0160:err:asan:asan_report stack-buffer-overflow, addr is 4 bytes to the right of the end of stack 015c:0160:err:asan:asan_report stack: [015A8820, 015A8860) 015c:0160:err:asan:asan_report stack pc: 6866252C (IDispatchEx_RemoteInvokeEx_Proxy, dlls/dispex/i386-windows/disp_ex_p.c:54,1), descr: 2 32 4 10 _RetVal:55 48 44 11 __params:69 ``` the parameters info: ``` 0024:trace:rpc:ndr_client_call INITOUT 0024:trace:rpc:client_do_args param[0]: 015EB834 type 08 IsIn IsBasetype 0024:trace:rpc:client_do_args param[1]: 015EB838 type 08 IsIn IsBasetype 0024:trace:rpc:client_do_args param[2]: 015EB83C type 08 IsIn IsBasetype 0024:trace:rpc:client_do_args param[3]: 015EB840 type 16 MustSize MustFree IsIn IsSimpleRef 0024:trace:rpc:client_do_args param[4]: 015EB844 type b4 MustSize MustFree IsOut IsSimpleRef ServerAllocSize = 16 0024:trace:rpc:client_do_args param[5]: 015EB848 type 1a MustSize MustFree IsOut IsSimpleRef ServerAllocSize = 32 0024:trace:rpc:client_do_args param[6]: 015EB84C type 2f MustSize MustFree IsIn 0024:trace:rpc:client_do_args param[7]: 015EB850 type 08 IsIn IsBasetype 0024:trace:rpc:client_do_args param[8]: 015EB854 type 1b MustSize MustFree IsIn IsSimpleRef 0024:trace:rpc:client_do_args param[9]: 015EB858 type 21 MustSize MustFree IsIn IsOut IsSimpleRef 0024:trace:rpc:client_do_args param[10]: 015EB85C type 08 IsOut IsReturn IsBasetype ``` reading of `param[10]` at `015EB85C`, which is the non-existent `_RetVal`, triggers the asan report. -- v2: rpcrt4: Don't read past the end of params in client_do_args. https://gitlab.winehq.org/wine/wine/-/merge_requests/8639

3 2

Re: [PATCH v2 0/1] MR8532: winegstreamer: Force h264parse to output 'au' aligned byte-stream.
by Brendan McGrath (＠redmcg) 31 Jul '25

31 Jul '25

On Thu Jul 31 18:58:30 2025 +0000, Tim Clem wrote: > Ah, interesting. Thanks for looking in to that. I can't speak to any of > that, really - I'm not very familiar with winegstreamer at all. I was > more wondering if we needed to explicitly set `byte-stream`, and maybe > we don't?: tclem/wine!8. Not sure if the tests cover the initial issue > you were addressing though. And I guess if it fixates to AVC but doesn't > have the required `codec_data` then maybe `byte-stream` is the only > option. But like I said, that has broken h.264 on macOS. Yeah, sorry. I took a look yesterday in the hope of finding answers, but instead came away with more questions. But, as you probably already noticed, we do need that `stream-format=byte-stream` for `avdec_h264` to work. Otherwise we get a bunch of failed tests in `mf:transform`: https://gitlab.winehq.org/tclem/wine/-/jobs/178373#L2122 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8532#note_111763

1 0

[PATCH 0/5] MR8540: kernelbase: Handle reparse points in some high-level functions, part 1/2.
by Elizabeth Figura (＠zfigura) 31 Jul '25

31 Jul '25

These do not do anything currently, since we don't support FILE_OPEN_REPARSE_POINT on the Unix side yet. However, they are correct, as demonstrated by the recently committed tests, and it seems prudent to get these out of the way for now. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8540

4 9

[PATCH v2 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. -- v2: dsound: Improve IDirectSoundBufferImpl_Lock handling of invalid arguments. dsound/tests: Add tests for IDirectSoundBuffer_Lock. https://gitlab.winehq.org/wine/wine/-/merge_requests/8632

3 4

[PATCH 0/1] MR8504: winemac: Get rid of unnecessary child window views.
by Rémi Bernon 31 Jul '25

31 Jul '25

I think this dates from a time where every window had its own window surface, and unless I'm missing something this isn't needed anymore. My intention is to refactor the client surfaces, so that they can be manipulated directly from win32u, and unify GL/VK client surfaces. This would make the code simpler, in preparation for that. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/8504

2 2