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

[PATCH v2 0/1] MR423: ieframe fix
by David Kahurani (＠kahurani) 11 Jul '22

11 Jul '22

Signed-off-by: David Kahurani <k.kahurani(a)gmail.com> -- v2: ieframe: Fix memory leak and possible reference leak https://gitlab.winehq.org/wine/wine/-/merge_requests/423

4 4

[PATCH 0/1] MR423: ieframe fix
by David Kahurani (＠kahurani) 11 Jul '22

11 Jul '22

Signed-off-by: David Kahurani <k.kahurani(a)gmail.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/423

3 2

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) 11 Jul '22

11 Jul '22

Jinoh Kang (@iamahuman) commented about dlls/kernelbase/file.c: > */ > BOOL WINAPI DECLSPEC_HOTPATCH CancelSynchronousIo( HANDLE thread ) > { > - FIXME( "(%p): stub\n", thread ); > - SetLastError( ERROR_CALL_NOT_IMPLEMENTED ); > - return FALSE; > + IO_STATUS_BLOCK io; > + > + return set_ntstatus( NtCancelSynchronousIoFile( thread, NULL, &io ) ); Per Wine code style, you should omit the second space in `&io ) )`. ```suggestion:-0+0 return set_ntstatus( NtCancelSynchronousIoFile( thread, NULL, &io )); ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3876

1 0

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) 11 Jul '22

11 Jul '22

Jinoh Kang (@iamahuman) commented about dlls/kernel32/tests/pipe.c: > + > + SetLastError(0xdeadbeef); > + res = pCancelSynchronousIo(GetCurrentThread()); > + ok(!res, "CancelSynchronousIo succeeded unexpectedly\n"); > + todo_wine > + ok(GetLastError() == ERROR_NOT_FOUND, > + "In CancelSynchronousIo failure, expected ERROR_NOT_FOUND, got %ld\n", GetLastError()); > + > + /* synchronous i/o */ > + pipe = CreateNamedPipeA(PIPENAME, PIPE_ACCESS_DUPLEX, > + PIPE_TYPE_MESSAGE | PIPE_READMODE_MESSAGE | PIPE_WAIT, > + 1, 1024, 1024, NMPWAIT_USE_DEFAULT_WAIT, NULL); > + ok(pipe != INVALID_HANDLE_VALUE, "CreateNamedPipe failed with %lu\n", GetLastError()); > + thread = CreateThread(NULL, 0, synchronousIoThreadMain, pipe, 0, NULL); > + /* wait for thread to start listening */ > + Sleep(100); How about the following race-free approach? ```suggestion:-0+0 /* wait for I/O to start, which transitions the pipe handle from signaled to nonsignaled state. */ while ((wait_status = WaitForSingleObject(pipe, 0)) == WAIT_OBJECT_0) Sleep(1); ok(wait_status == WAIT_TIMEOUT, "WaitForSingleObject returned %lu (error %lu)\n", wait_status, GetLastError()); ``` Ditto for other places. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3874

1 0

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) 11 Jul '22

11 Jul '22

Jinoh Kang (@iamahuman) commented about server/async.c: > return woken; > } > > +static int cancel_blocking( struct process *process, struct thread *thread, client_ptr_t iosb ) > +{ > + struct async *async; > + int woken = 0; > + > +restart: > + LIST_FOR_EACH_ENTRY( async, &process->asyncs, struct async, process_entry ) > + { > + if (async->terminated || async->canceled) continue; > + if (async->blocking && (async->thread == thread) && Maybe you forgot to remove the extraneous parentheses? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3875

1 0

Re: [PATCH v2 0/4] MR47: kernel32/tests: Add tests for CancelSynchronousIo.
by Jinoh Kang (＠iamahuman) 11 Jul '22

11 Jul '22

> the test would hang on wine before the implementation is in place. Thanks! I missed that the current test cannot verify the Windows behaviour before the commit that introduces the Wine implementation. > the CloseHandle before the WaitForSingleObject is a workaround to get the thread to exit I'm afraid that closing a handle in use is racy by nature; it's effectively pulling the rug out from underneath. A better approach would be to use `CancelIoEx` on the `OVERLAPPED` structure, or `NtCancelIoFileEx` on the `IO_STATUS_BLOCK` structure to cancel the pending operation. > > moving the CloseHandle was meant to show that workaround is no longer needed and that it now performs like windows > > i can remove that if preferred -- https://gitlab.winehq.org/wine/wine/-/merge_requests/47#note_3873

1 0

[PATCH v3 0/2] MR360: Draft: winepulse.drv: add support for containerId property from sysfs path
by Claire (＠ClearlyClaire) 11 Jul '22

11 Jul '22

Some games with support for the haptic feedback and speaker features of the Sony DualSense controller select the controller's audio output by filtering on the ContainerId IMMDevice property to find one that matches the controller's HID's. This MR, together with !359, adds support for exposing such a ContainerId to applications. I marked this MR as a draft because I understand that the way the ��guid�� is generated is far from ideal. Furthermore, I will need to map from sysfs to container Id in other components as well (`winebus` for the HID device, and possibly `winealsa` and other audio drivers), so moving that part elsewhere would make sense. However, I think I will need help with those tasks. -- v3: winepulse: Add support for containerId property from sysfs path winepulse: Store PulseAudio device's sysfs path when available https://gitlab.winehq.org/wine/wine/-/merge_requests/360

2 2

[PATCH v4 0/4] MR359: mmdevapi: copy ContainerID property from audio driver if available
by Claire (＠ClearlyClaire) 11 Jul '22

11 Jul '22

Some games with support for the haptic feedback and speaker features of the Sony DualSense controller select the controller's audio output by filtering on the ContainerId IMMDevice property to find one that matches the controller's HID's. This MR allows this information to be accessible from the IMMDevice's property store when the audio driver provides it (none for now, but I intend to add that feature to `winepulse.drv` and maybe `winealsa.drv`) This is made specific to containerId rather than supporting VT_CLSID on every property because Wine currently stores VT_BLOBs directly in the registry value, which does not allow us to safely disambiguate between VT_CLSID and VT_BLOB values when reading from registry. -- v4: mmdevapi: Invalidate ContainerID of unavailable audio devices mmdevapi: copy ContainerID from audio driver if available mmdevapi: decode ContainerId property to CLSID in MMDevice_GetPropValue mmdevapi: support VT_CLSID for containerId property in MMDevice_SetPropValue https://gitlab.winehq.org/wine/wine/-/merge_requests/359

3 4

[PATCH 0/6] MR420: imm32: win32u conversion
by Jacek Caban (＠jacek) 11 Jul '22

11 Jul '22

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

3 7

[PATCH 0/1] MR419: user32/tests: Check that a message window brought topmost does not defocus other windows.
by Giovanni Mascellani (＠giomasce) 11 Jul '22

11 Jul '22

libOVR brings a message window topmost claiming that this way it receives WM_DEVICECHANGE faster. Currently, in Wine, this causes a WM_KILLFOCUS to be sent to the actual topmost window, which in turn causes "Shantae: Risky's Revenge" to minimize. The effect is that the game automatically minimizes as soon as it is launched, which is incorrect. Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/419

2 1