January 2025 - Wine-GitLab - list.winehq.org

[PATCH 0/1] MR7134: kernel32/tests: Avoid stack-buffer-overflow in get_com_dir_size. (ASan)
by Nikolay Sivov (＠nsivov) Jan. 12, 2025

Jan. 12, 2025

Function gets called with Magic == 0xdead, which causes the test to use the 64-bit path, which results in this buffer overflow. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7134

2 1

[PATCH 0/2] MR7133: gdiplus: Fix some path functions not starting a new figure.
by Esme Povirk (＠madewokherd) Jan. 11, 2025

Jan. 11, 2025

Found by Mono's System.Drawing tests. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7133

3 3

[PATCH 0/1] MR7132: gdiplus/tests: Trace locked bitmap data on failure.
by Esme Povirk (＠madewokherd) Jan. 11, 2025

Jan. 11, 2025

For debugging test failures like this one: https://test.winehq.org/data/8331e7db0d540656a047f415184a30df5acbc3f9/linux… -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7132

2 1

[PATCH v2 0/1] MR7055: appwiz.cpl: Account for unicode characters in XDG_CACHE_HOME.
by Esme Povirk (＠madewokherd) Jan. 11, 2025

Jan. 11, 2025

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=57338 -- v2: appwiz.cpl: Account for unicode characters in XDG_CACHE_HOME. https://gitlab.winehq.org/wine/wine/-/merge_requests/7055

2 2

Re: [PATCH v4 0/1] MR7091: windowscodecs: Make values returned from CanConvert consistent - approved
by Esme Povirk (＠madewokherd) Jan. 11, 2025

Jan. 11, 2025

This merge request was approved by Esme Povirk. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7091

1 0

Re: [PATCH v2 0/0] MR7130: (INVALID) - closed
by Jinoh Kang (＠iamahuman) Jan. 11, 2025

Jan. 11, 2025

This merge request was closed by Jinoh Kang. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7130

1 0

Re: [PATCH v2 0/7] MR6911: Draft: ntdll: Add WaitCompletionPacket.
by Jinoh Kang (＠iamahuman) Jan. 11, 2025

Jan. 11, 2025

Jinoh Kang (@iamahuman) commented about server/thread.c: > + { > + struct wait_completion_packet *packet; > + > + LIST_FOR_EACH_ENTRY( packet, &obj->wait_completion_packet_queue, struct wait_completion_packet, entry ) > + { > + list_remove( &packet->entry ); > + release_object( packet->target ); > + packet->in_object_packet_queue = 0; > + packet->target = NULL; > + > + add_completion( packet->completion, packet->ckey, packet->cvalue, packet->status, > + packet->information, packet ); > + packet->in_completion_queue = 1; > + } > + } > + General advice: you should try not to leak implementation details of certain object (wait completion packet) into other types of objects. For example, if this piece of code is still to be preserved, you should move this into a helper completion.c, declare as extern in object.h (or file.h), and call that. That way, you don't have to define the entire `struct wait_completion_packet` in a shared header file. If this were factored into a helper, a simple forward declaration would have sufficed in the `file.h` side. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6911#note_91785

1 0

[PATCH 0/1] MR7129: ntoskrnl.exe: Fix IRQL mismatch between cancel spin lock acquire and release.
by Jinoh Kang (＠iamahuman) Jan. 11, 2025

Jan. 11, 2025

Use the irql assigned by IoAcquireCancelSpinLock, not the (yet-to-be-assigned) irp->CancelIrql. This is for readability. No semantic change is expected with unmodified ntoskrnl.exe/hal.dll. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7129

3 2

[PATCH v2 0/1] MR7129: ntoskrnl.exe: Fix IRQL mismatch between cancel spin lock acquire and release.
by Jinoh Kang (＠iamahuman) Jan. 11, 2025

Jan. 11, 2025

Use the irql assigned by IoAcquireCancelSpinLock, not the (yet-to-be-assigned) irp->CancelIrql. This is for readability. Since IoReleaseCancelSpinLock() ignores the IRQL argument anyway, no semantic change is expected with unmodified ntoskrnl.exe/hal.dll. -- v2: ntoskrnl.exe: Fix IRQL mismatch between cancel spin lock acquire and release. https://gitlab.winehq.org/wine/wine/-/merge_requests/7129

2 1

Re: [PATCH v2 0/1] MR7124: mfmediaengine: Fix maths in scaling check. - approved
by Alexandre Julliard (＠julliard) Jan. 10, 2025

Jan. 10, 2025

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7124

1 0