[Bug 53712] New: ole32:moniker is broken on Windows 7 in the GitLab WineTest builds

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=53712

Bug ID: 53712
           Summary: ole32:moniker is broken on Windows 7 in the GitLab
                    WineTest builds
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: ole32
          Assignee: [e-mail]
          Reporter: [e-mail]

ole32:moniker is broken on Windows 7 in the GitLab WineTest builds:

moniker.c:2280: ACP is 1252
0a18:4þ": 5504 tests executed (0 marked as todo, 0 as flaky, 0 failures), 0
skipped.
ole32:moniker:0a18 done (-1073740791) in 0s 115B
Test failed: crash (c0000409)

https://test.winehq.org/data/patterns.html#ole32:moniker

Where 0xc0000409 == STATUS_STACK_BUFFER_OVERRUN.

The failures started on 2022-07-26 which is when the WineTest builds switched
from the old build system to GitLab. A test with winetest-889fe64d9f2e.exe
(built on 2022-07-25) confirms that this test still succeeds when using
binaries from the old build system.

[Bug 53712] ole32:moniker is broken on Windows 7 in the GitLab WineTest builds

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=53712

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |source, testcase
                 CC|                            |[e-mail]

[Bug 53712] ole32:moniker is broken on Windows 7 in the GitLab WineTest builds

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=53712

--- Comment #1 from François Gouget  ---
Created attachment 74166
  --> https://bugs.winehq.org/attachment.cgi?id=74166
Slim down ole32:moniker for the w7u crash.

The test does not always crash so one should also look for the mangled test
unit name on the test summary line: « 0a18:4þ": ».

Converting any of the "if (1)" to an "if (0)" gets rid of any visible trace of
memory corruption.

[Bug 53712] ole32:moniker is broken on Windows 7 in the GitLab WineTest builds

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=53712

--- Comment #2 from François Gouget  ---
The crash code changed to c0000005 == STATUS_ACCESS_VIOLATION on 2023-07-11 (so
not on the  day of the GitLab CI switch to Debian 12, but the day after).

[Bug 53712] ole32:moniker is broken on Windows 7 in the GitLab WineTest builds

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=53712

--- Comment #3 from François Gouget  ---
On 2023-08-01 w7u-pt-PT got a STATUS_ILLEGAL_INSTRUCTION crash on exit instead
but it's probably still the same issue:

0a74:¤ö9: 5505 tests executed (0 marked as todo, 0 as flaky, 0 failures), 0
skipped.
ole32:moniker:0a74 done (-1073741795) in 0s 115B
Test failed: crash (c000001d)