[Bug 54657] New: kernel32:loader - test_import_resolution() gets bad tls data on Windows 7

[WineHQ Bugzilla]

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

Bug ID: 54657
           Summary: kernel32:loader - test_import_resolution() gets bad
                    tls data on Windows 7
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: kernel32
          Assignee: [e-mail]
          Reporter: [e-mail]

kernel32:loader - test_import_resolution() gets bad tls data on Windows 7:

loader.c:2324: Test failed: wrong tls data 'hello world' at 0000000000320590

See https://test.winehq.org/data/patterns.html#kernel32:loader

[Bug 54657] kernel32:loader - test_import_resolution() gets bad tls data on Windows 7

[WineHQ Bugzilla]

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

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |source, testcase

[Bug 54657] kernel32:loader - test_import_resolution() gets bad tls data on Windows 7

[WineHQ Bugzilla]

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

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
           Keywords|                            |regression

--- Comment #1 from François Gouget  ---
This test failure was introduced in the commit below:

commit 2203a8564c5faba383a3512f8244dba12d79da16
Author:     Evan Tang 
AuthorDate: Wed Mar 8 16:57:43 2023 -0600

kernel32/tests: Add test verifying that tls init functions are called.

Note that this failure was clearly reported in MR2361 but the TestBot report
was ignored apparently.
https://gitlab.winehq.org/wine/wine/-/merge_requests/2361

[Bug 54657] kernel32:loader - test_import_resolution() gets bad tls data on Windows 7

[WineHQ Bugzilla]

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

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |a7b49b0e86502125214c3672f56
                   |                            |ecda7845dcb0d

--- Comment #2 from François Gouget  ---
Yay, this is fixed:

commit a7b49b0e86502125214c3672f56ecda7845dcb0d
Author: Evan Tang 
Date:   Fri Mar 10 20:39:38 2023 -0600

kernel32/tests: Fix tls callback tests on Windows 7.

Also add a few more so the full set of callbacks is tested.

Fixes: 2203a8564c5faba383a3512f8244dba12d79da16
    Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54657

[Bug 54657] kernel32:loader - test_import_resolution() gets bad tls data on Windows 7

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #3 from Alexandre Julliard  ---
Closing bugs fixed in 8.4.