[Bug 36661] New: valgrind shows an unintialized write in ws2_32/tests/sock.c: test_completion_port()

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

Bug ID: 36661
           Summary: valgrind shows an unintialized write in
                    ws2_32/tests/sock.c: test_completion_port()
           Product: Wine
           Version: 1.3.30
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase, valgrind
          Severity: normal
          Priority: P2
         Component: winsock
          Assignee: [e-mail]
          Reporter: [e-mail]

==21116== Syscall param sendmsg(msg.msg_iov[0]) points to uninitialised byte(s)
==21116==    at 0x4226288: sendmsg (socket.S:98)
==21116==    by 0x50A9CF8: WS2_send (socket.c:2178)
==21116==    by 0x50B08E0: WS2_sendto (socket.c:4569)
==21116==    by 0x50B0611: WSASend (socket.c:4500)
==21116==    by 0x4F7ACDD: test_completion_port (sock.c:6929)
==21116==    by 0x4F7FDAC: func_sock (sock.c:7739)
==21116==    by 0x4F80BB1: run_test (test.h:584)
==21116==    by 0x4F80FA0: main (test.h:654)
==21116==  Address 0x508f89c is on thread 1's stack
==21116==  Uninitialised value was created by a stack allocation
==21116==    at 0x4F7A37C: test_completion_port (sock.c:6820)
==21116==

==21116== Syscall param sendmsg(msg.msg_iov[0]) points to uninitialised byte(s)
==21116==    at 0x4226288: sendmsg (socket.S:98)
==21116==    by 0x50A9CF8: WS2_send (socket.c:2178)
==21116==    by 0x50B08E0: WS2_sendto (socket.c:4569)
==21116==    by 0x50B05AF: WS_send (socket.c:4486)
==21116==    by 0x4F7E05B: test_completion_port (sock.c:7347)
==21116==    by 0x4F7FDAC: func_sock (sock.c:7739)
==21116==    by 0x4F80BB1: run_test (test.h:584)
==21116==    by 0x4F80FA0: main (test.h:654)
==21116==  Address 0x508f89c is on thread 1's stack
==21116==  Uninitialised value was created by a stack allocation
==21116==    at 0x4F7A37C: test_completion_port (sock.c:6820)
==21116==

[Bug 36661] valgrind shows an unintialized write in ws2_32/tests/sock.c: test_completion_port()

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|1.3.30                      |1.7.19

[Bug 36661] valgrind shows an unintialized write in ws2_32/tests/sock.c: test_completion_port()

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

--- Comment #1 from Bruno Jesus  ---
Created attachment 48714
  --> https://bugs.winehq.org/attachment.cgi?id=48714
patch

Please test the attached patch.

[Bug 36661] valgrind shows an unintialized write in ws2_32/tests/sock.c: test_completion_port()

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

--- Comment #2 from Austin English  ---
(In reply to Bruno Jesus from comment #1)
> Created attachment 48714 [details]
> patch
>
> Please test the attached patch.

Works for me, thanks.

[Bug 36661] valgrind shows an unintialized write in ws2_32/tests/sock.c: test_completion_port()

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |79d29e2347b7d1780bc1b71f2dd
                   |                            |6c64e564f6e61
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Austin English  ---
(In reply to Bruno Jesus from comment #1)
> Created attachment 48714 [details]
> patch
>
> Please test the attached patch.

https://source.winehq.org/git/wine.git/commitdiff/79d29e2347b7d1780bc1b71f2d...

[Bug 36661] valgrind shows an unintialized write in ws2_32/tests/sock.c: test_completion_port()

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

Alexandre Julliard  changed:

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

--- Comment #4 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.20.