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

Re: [PATCH v4 0/1] MR7925: gdi32: Limit source string length in logfont_AtoW (ASan).
by Rémi Bernon 02 May '25

02 May '25

On Fri May 2 16:52:15 2025 +0000, Alex Henrie wrote: > I apologize for the stupid mistake. !7956 will fix it. Literally two > minutes after I sent it, Bernhard sent an alternative fix in !7957 which > would also be fine. > I recognize that this happened primarily due to human error on my part. > Nevertheless, another part of the problem is that the 32-bit tests > always fail, and somehow this mistake did not cause the 64-bit tests to > fail. That means that I need to spend some time helping with the > intermittent test failures because they're making it too easy for me to > overlook test failures that actually matter. Until the tests can be > expected to pass, I will have to remember to read through the entire > list of failures whenever I send a patch. The test-linux-64 test suite only run a few tests that are otherwise skipped in a old/new wow64 environment like the one used for test-linux-32. It's maybe a bit confusing but it otherwise would use another runner from >30min and it wasn't deemed useful to have a full test coverage run on every config. The test-linux-32 sadly fails often, spuriously, although there have been up and downs I think it's getting better now and hopefully we'll get back to some occasional failures only. Nevertheless, Gitlab now shows a short summary of failing tests (in the MRs "Test summary:" section, which can be expanded for a detailed list of failures) and I think we should be able to quickly check MR failures, and compare with other MRs or the testbot pattern page, to decide whether they are likely related or not. Usually there's only a handful of spurious failures, with some well known returning failures, and a large number of/unusual failures should already still stand out, if we're paying attention. Fwiw I'm not blaming you specifically here, it happens quite often (and I can be blamed for a fair amount the last few months). I'm mostly making sure it doesn't go unnoticed and try to raise awareness. At least as long as things get caught and fixed quickly it's probably fine. Fixing spurious failures would be nice too but that's much more difficult than it seems. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7925#note_102379

1 0

Re: [PATCH v4 0/1] MR7925: gdi32: Limit source string length in logfont_AtoW (ASan).
by Alex Henrie (＠alexhenrie) 02 May '25

02 May '25

On Fri May 2 16:52:15 2025 +0000, Rémi Bernon wrote: > This broke comctl32:status dwrite:font (win32/wow64) gdi32:font > user32:sysparams usp10:usp10 tests. I apologize for the stupid mistake. !7956 will fix it. Literally two minutes after I sent it, Bernhard sent an alternative fix in !7957 which would also be fine. I recognize that this happened primarily due to human error on my part. Nevertheless, another part of the problem is that the 32-bit tests always fail, and somehow this mistake did not cause the 64-bit tests to fail. That means that I need to spend some time helping with the intermittent test failures because they're making it too easy for me to overlook test failures that actually matter. Until the tests can be expected to pass, I will have to remember to read through the entire list of failures whenever I send a patch. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7925#note_102376

1 0

[PATCH v3 0/2] MR7780: win32u: Avoid uneeded (and sometimes harmful) RedrawWindow() calls in expose_window_surface().
by Paul Gofman (＠gofman) 02 May '25

02 May '25

-- v3: win32u: Don't inflate rect in expose_window_surface(). win32u: Don't redraw window in expose_window_surface() if window has surface. https://gitlab.winehq.org/wine/wine/-/merge_requests/7780

3 7

[PATCH 0/1] MR7954: msvcrt: Add truncf to the import library.
by Stefan Dösinger 02 May '25

02 May '25

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

3 3

[PATCH v4 0/4] MR7934: mshtml: Validate builtin host functions in IE9+ using prototype_id rather than tid where possible.
by Gabriel Ivăncescu 02 May '25

02 May '25

-- v4: mshtml: Remove unused struct mutation_observer_ctor. mshtml: Use designated initializers for the Location dispex data. mshtml: Validate builtin host functions in IE9+ using prototype_id rather mshtml: Store the object_id of the last object in the prototype chain that https://gitlab.winehq.org/wine/wine/-/merge_requests/7934

2 5

[PATCH 0/4] MR7955: secur32/tests: Fix NTLM tests on Windows 11.
by Piotr Caban (＠piotr) 02 May '25

02 May '25

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

2 4

[PATCH 0/1] MR7947: ntdll: Make sure NT flag is not set before iretq in wine_syscall_dispatcher_return on x64.
by Paul Gofman (＠gofman) 02 May '25

02 May '25

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

3 4

[PATCH 0/1] MR7952: msv1_0: Drop the ntlm_auth check.
by Hans Leidekker (＠hans) 02 May '25

02 May '25

This means that the diagnostic message is no longer shown when secur32 is loaded. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7952

3 7

[PATCH 0/6] MR7953: win32u: Add a generic wgl(Set|Get)SwapInterval implementation.
by Rémi Bernon 02 May '25

02 May '25

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

1 6

Re: [PATCH v4 0/1] MR7925: gdi32: Limit source string length in logfont_AtoW (ASan).
by Rémi Bernon 02 May '25

02 May '25

This broke comctl32:status dwrite:font (win32/wow64) gdi32:font user32:sysparams usp10:usp10 tests. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7925#note_102334

1 0