[Bug 47032] New: valgrind reports uninitialized read in writev from call_req_handler

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

Bug ID: 47032
           Summary: valgrind reports uninitialized read in writev from
                    call_req_handler
           Product: Wine
           Version: 4.6
          Hardware: x86
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Created attachment 64201
  --> https://bugs.winehq.org/attachment.cgi?id=64201
Output when running main.exe under valgrind

Starting with cloning the source tree

$ git clone git://source.winehq.org/git/wine.git

$ mkdir wine64_build
    $ cd wine64_build
    $ ../wine/configure --enable-win64
    $ make
    $ cd ..

$ mkdir test_program
    $ cd test_program
    $ echo 'int main() { return 0; }' > main.c
    $ x86_64-w64-mingw32-gcc main.c -o main.exe
    $ valgrind --trace-children=yes ../wine64_build/loader/wine64 ./main.exe

gives the attached output.  The first error appears to be an uninitialized read
to writev in request.c (even though it's hard to tell since valgrind is tracing
multiple threads).

[Bug 47032] valgrind shows uninitialized read in writev from call_req_handler

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

[e-mail] changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|valgrind reports            |valgrind shows
                   |uninitialized read in       |uninitialized read in
                   |writev from                 |writev from
                   |call_req_handler            |call_req_handler

[Bug 47032] valgrind shows uninitialized read in writev from call_req_handler

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

--- Comment #1 from [e-mail] ---
This occurs because the blob mapping->image is set as the reply data, however
that struct (pe_image_info_t) contains 4 uninitialized padding bytes.

[Bug 47032] valgrind shows uninitialized read in writev from call_req_handler

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

Józef Kucia  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|-unknown                    |wineserver
      Fixed by SHA1|                            |dd318b135d359cc6ea558214b32
                   |                            |122ef42d21a28
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Józef Kucia  ---
Fixed by
https://source.winehq.org/git/wine.git/?a=commit;h=dd318b135d359cc6ea558214b32122ef42d21a28.

[Bug 47032] valgrind shows uninitialized read in writev from call_req_handler

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

Alexandre Julliard  changed:

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

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