[Bug 49484] New: test_query_process_image_info() test is broken on a not PE build

[WineHQ Bugzilla]

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

Bug ID: 49484
           Summary: test_query_process_image_info() test is broken on a
                    not PE build
           Product: Wine
           Version: 5.11
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: regression
          Severity: normal
          Priority: P2
         Component: -unknown
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Steps to reproduce:

cd wine/dlls/ntdll/tests
make info.ok

info.c:1843: Test failed: got c0000019
info.c:1846: Test failed: wrong major version cccc/4
info.c:1848: Test failed: wrong minor version cccc/0

In fact it never worked from the point when it was added:

commit 54a7e592e44622ecd61160498fb5148a46b2e8db
Author: Alexandre Julliard 
Date:   Tue May 26 13:23:21 2020 +0200

ntdll: Implement ProcessImageInformation class in
NtQueryInformationProcess().

It looks like in a not PE build process->views on the server side is always
empty, and as a result mapping.c,find_mapped_view() returns NOT_MAPPED_VIEW.
There's no a single DLL in the process->views, and that's a real regression,
probably caused by the movement to PE/ntdll unixlib introduction.

[Bug 49484] test_query_process_image_info() test is broken on a not PE build

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|regression                  |

--- Comment #1 from Alexandre Julliard  ---
I don't think it's a regression, we've never created views for .so dlls. It's
not clear how we would do it since they don't have a corresponding mapping.

[Bug 49484] test_query_process_image_info() test is broken on a not PE build

[WineHQ Bugzilla]

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

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |ced3892c65b33149a490b2c2191
                   |                            |098c0c6aa1eeb
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #2 from Dmitry Timoshkov  ---
Fixed by ced3892c65b33149a490b2c2191098c0c6aa1eeb.

Thanks Alexandre.

[Bug 49484] test_query_process_image_info() test is broken on a not PE build

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

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

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