March 2025 - wine-gitlab

[PATCH 0/2] MR7537: setupapi: Fix id size handling in get_device_id_list().
by Paul Gofman (＠gofman) 12 Mar '25

12 Mar '25

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

3 5

[PATCH 0/3] MR6925: d3dcompiler: Enable double-as-float option for older modules.
by Nikolay Sivov (＠nsivov) 12 Mar '25

12 Mar '25

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55448 Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6925

4 14

[PATCH v2 0/1] MR7560: Draft: gdiplus/tests: Extend millimiter test for GdipIsOutlineVisiblePathPoint.
by Bartosz Kosiorek (＠gang65) 12 Mar '25

12 Mar '25

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55717 -- v2: gdiplus/tests: Extend millimiter test for GdipIsOutlineVisiblePathPoint. https://gitlab.winehq.org/wine/wine/-/merge_requests/7560

2 1

[PATCH 0/1] MR7560: Draft: gdiplus/tests: Extend millimiter test for GdipIsOutlineVisiblePathPoint.
by Bartosz Kosiorek (＠gang65) 12 Mar '25

12 Mar '25

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55717 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7560

2 1

[PATCH 0/1] MR7551: win32u: Prevent remote drawing to ULW layered window.
by Paul Gofman (＠gofman) 12 Mar '25

12 Mar '25

Fixes a regression introduced by 309daa5beef262333b8c5328018782401c3380a6 . -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7551

3 4

[PATCH v3 0/1] MR7357: rpcrt4: Avoid unsigned overflows in buffer validation.
by Jacek Caban (＠jacek) 12 Mar '25

12 Mar '25

Fixes Clang 20 `-Wtautological-compare` warnings. Based on patch by William Horvath. -- v3: rpcrt4: Don't validate buffer in NDR marshaler. https://gitlab.winehq.org/wine/wine/-/merge_requests/7357

3 2

Re: [PATCH v27 0/1] MR6884: Iphlpapi.dll: Included SendARP
by Huw Davies (＠huw) 12 Mar '25

12 Mar '25

On Tue Mar 11 14:54:20 2025 +0000, Thi Garlet wrote: > @huw, looking at > [netioapi](https://gitlab.winehq.org/wine/wine/-/blob/master/include/netioap…, > I found some functions from [IP Helper > overview](https://learn.microsoft.com/en-us/windows-hardware/drivers/networ…, > but ResolveIpNetEntry2() is not there. > The SendARP() is workig, is something missing to merge ? > thanks Well the idea is you'd implement `ResolveIpNetEntry2()` and have `SendARP()` call it. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6884#note_97547

1 0

[PATCH 0/5] MR7511: opengl32: Stop generating wine/opengl_driver.h.
by Rémi Bernon (＠rbernon) 12 Mar '25

12 Mar '25

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

5 8

[PATCH 0/1] MR7558: kernel32/tests: Flush pending APCs and close handles (ASan).
by Bernhard Übelacker (＠bernhardu) 12 Mar '25

12 Mar '25

ASan gets triggered in `test_read_write` because an APC gets executed while `test_async_file_errors` is already left, and therefore the `ovl` variable in is no longer at the stack. This patch tries to flush the APC before leaving the function `test_async_file_errors`. Also closes two handles opened inside this function. ``` ================================================================= ==508==ERROR: AddressSanitizer: stack-use-after-scope on address 0x7ffffe1ea7f8 at pc 0x0001401a4933 bp 0x7ffffe1e2a80 sp 0x7ffffe1e2ac8 READ of size 8 at 0x7ffffe1ea7f8 thread T0 0208:fixme:file:server_get_file_info Unsupported info class e #0 0x0001401a4932 in FileIOComplete .../wine/dlls/kernel32/tests/file.c:3337:24 #1 0x6ffffacef75f in KiUserApcDispatcher (C:\windows\system32\ntdll.dll+0x17000f75f) #2 0x6ffffacedf53 in NtDelayExecution (C:\windows\system32\ntdll.dll+0x17000df53) #3 0x6ffffa6eec5f in SleepEx .../wine/dlls/kernelbase\sync.c:346:14 #4 0x00014015a0e3 in test_read_write .../wine/dlls/kernel32/tests/file.c:3451:5 #5 0x00014015a0e3 in func_file .../wine/dlls/kernel32/tests/file.c:6348:5 #6 0x0001404d7fa1 in run_test .../wine/include/wine/test.h:765:5 #7 0x0001404d7fa1 in main .../wine/include/wine/test.h:884:12 #8 0x0001404d9f8f in mainCRTStartup .../wine/dlls/msvcrt/crt_main.c:58:11 #9 0x6ffffbfd4808 in BaseThreadInitThunk .../wine/dlls/kernel32\thread.c:61:5 #10 0x6ffffacefa1a in RtlUserThreadStart (C:\windows\system32\ntdll.dll+0x17000fa1a) Address 0x7ffffe1ea7f8 is located in stack of thread T0 at offset 30232 in frame #0 0x00014013223f in func_file .../wine/dlls/kernel32/tests/file.c:6296 This frame has 205 object(s): [32, 292) 'cwd.i6426' (line 6001) [368, 628) 'temp_dir.i6427' (line 6001) [704, 964) 'cwd.i' (line 5922) ... [29536, 29796) 'filename.i2304' (line 3395) [29872, 30132) 'szFile.i' (line 3343) [30208, 30240) 'ovl.i' (line 3347) <== Memory access at offset 30232 is inside this variable [30272, 30276) 'count.i' (line 3364) [30288, 30296) 'h.i' (line 3246) ... [116896, 117156) 'temp_path' (line 6297) HINT: this may be a false positive if your program uses some custom stack unwind mechanism, swapcontext or vfork (longjmp, SEH and C++ exceptions *are* supported) SUMMARY: AddressSanitizer: stack-use-after-scope .../wine/dlls/kernel32/tests/file.c:3337:24 in FileIOComplete Shadow bytes around the buggy address: 0x7ffffe1ea500: f2 f2 f2 f2 f2 f2 f2 f2 00 00 00 00 00 00 00 00 0x7ffffe1ea580: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 0x7ffffe1ea600: 00 00 00 00 00 00 00 00 04 f2 f2 f2 f2 f2 f2 f2 0x7ffffe1ea680: f2 f2 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 0x7ffffe1ea700: f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 =>0x7ffffe1ea780: f8 f8 f8 f2 f2 f2 f2 f2 f2 f2 f2 f2 f8 f8 f8[f8] 0x7ffffe1ea800: f2 f2 f2 f2 f8 f2 f8 f2 f2 f2 f8 f2 f2 f2 f8 f8 0x7ffffe1ea880: f8 f8 f2 f2 f2 f2 f8 f2 f8 f8 f8 f8 f8 f8 f8 f8 0x7ffffe1ea900: f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 0x7ffffe1ea980: f8 f8 f8 f8 f8 f8 f8 f8 f2 f2 f2 f2 f2 f2 f2 f2 0x7ffffe1eaa00: f8 f2 f2 f2 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 f8 Shadow byte legend (one shadow byte represents 8 application bytes): Addressable: 00 Partially addressable: 01 02 03 04 05 06 07 Heap left redzone: fa Freed heap region: fd Stack left redzone: f1 Stack mid redzone: f2 Stack right redzone: f3 Stack after return: f5 Stack use after scope: f8 Global redzone: f9 Global init order: f6 Poisoned by user: f7 Container overflow: fc Array cookie: ac Intra object redzone: bb ASan internal: fe Left alloca redzone: ca Right alloca redzone: cb ==508==ABORTING ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7558

2 1

Re: [PATCH v3 0/1] MR7323: wininet: Treat Expires value "-1" the same as "0"
by Jacek Caban (＠jacek) 12 Mar '25

12 Mar '25

On Wed Mar 12 10:25:16 2025 +0000, Yongjie Yao wrote: > I add 5 tests to test_http_connection(). > The first test sends "Expires: 0", the second test sends "Expires: -1", > the third test sends "Expires: 100", the fourth test sends "Expires: > invalid date", the fifth test sends "Expires: Thu, 06 Mar 2025 06:08:11 GMT". > The tests on actual windows 10 has output informations, such as: > ```null > http.c:6837: expires: 2025-03-06 06:14:13, current time: 2025-03-06 > 06:14:13, server response expires: 0 > http.c:6837: expires: 2025-03-06 06:14:14, current time: 2025-03-06 > 06:14:14, server response expires: -1 > http.c:6837: expires: 2025-03-06 06:14:14, current time: 2025-03-06 > 06:14:14, server response expires: 100 > http.c:6837: expires: 2025-03-06 05:14:15, current time: 2025-03-06 > 06:14:15, server response expires: invalid date > http.c:6837: expires: 2025-03-06 06:08:11, current time: 2025-03-06 > 06:14:15, server response expires: Thu, 06 Mar 2025 06:08:11 GMT > ``` > I think Windows treats all numbers as invalid values and sets the > expiration time to the current system time, while other invalid values > set the expiration time to one hour before the current system time, and > valid expiration times use the set expiration time. > In Wine, handling inconsistencies with Windows 10， We treat "0" as > "1601-01-01 00:00:00", and all other invalid values as 10 minutes after > the current system time. such as > ```coffeescript > http.c:6837: expires: 1601-01-01 00:00:00, current time: 2025-03-06 > 06:10:56, server response expires: 0 > 0118:err:wininet:HTTP_ParseDateAsAsctime unexpected weekday L"-1" > 0118:err:wininet:HTTP_ParseDate unexpected date format L"-1" > http.c:6837: expires: 2025-03-06 06:20:56, current time: 2025-03-06 > 06:10:57, server response expires: -1 > 0110:err:wininet:HTTP_ParseDateAsAsctime unexpected weekday L"100" > 0110:err:wininet:HTTP_ParseDate unexpected date format L"100" > http.c:6837: expires: 2025-03-06 06:20:56, current time: 2025-03-06 > 06:10:57, server response expires: 100 > 0118:err:wininet:HTTP_ParseDateAsAsctime unexpected weekday L"inv" > 0118:err:wininet:HTTP_ParseDate unexpected date format L"invalid date" > http.c:6837: expires: 2025-03-06 06:20:56, current time: 2025-03-06 > 06:10:58, server response expires: invalid date > http.c:6837: expires: 2025-03-06 06:08:10, current time: 2025-03-06 > 06:10:58, server response expires: Thu, 06 Mar 2025 06:08:11 GMT > ``` Those are interesting findings, thanks. Could you please also check what's the behavior for missing "Expires" header? It sounds like we should change error handling in `HTTP_ProcessExpires`, I wonder if `!expirationFound` branch there is right... -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7323#note_97540

1 0

2025

2024

2023

2022

wine-gitlab March 2025