[Bug 36599] New: winhttp/winhttp.c test fails under valgrind

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

Bug ID: 36599
           Summary: winhttp/winhttp.c test fails under valgrind
           Product: Wine
           Version: 1.7.19
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: normal
          Priority: P2
         Component: winhttp
          Assignee: [e-mail]
          Reporter: [e-mail]

../../../tools/runtest -q -P wine -T ../../.. -M winhttp.dll -p
winhttp_test.exe.so winhttp && touch winhttp.ok
preloader: Warning: failed to reserve range 00110000-68000000
preloader: Warning: failed to reserve range 7f000000-82000000
fixme:winhttp:request_query_option partial stub

winhttp.c:879: Test failed: failed to receive response 12152
winhttp.c:883: Test failed: failed unexpectedly 12150
winhttp.c:884: Test failed: request failed unexpectedly 122
winhttp.c:899: Test failed: read_size = 0

causing a lot of valgrind warnings about uninitialized values in
winetest_ok/etc.

[Bug 36599] winhttp/tests/winhttp.c: test_secure_connection() fails under valgrind

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|winhttp/winhttp.c test      |winhttp/tests/winhttp.c:
                   |fails under valgrind        |test_secure_connection()
                   |                            |fails under valgrind

[Bug 36599] winhttp/tests/winhttp.c: test_secure_connection() fails under valgrind

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |valgrind

[Bug 36599] winhttp/tests/winhttp.c: test_secure_connection() fails under valgrind

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

--- Comment #1 from Austin English  ---
Still in 1.7.41.

[Bug 36599] winhttp/tests/winhttp.c: test_secure_connection() fails under valgrind

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

--- Comment #2 from Hans Leidekker  ---
This might be a problem with gnutls:

==29760== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 7 from 7)
==29749== Conditional jump or move depends on uninitialised value(s)
==29749==    at 0x61CE0A7: ??? (in /lib/i386-linux-gnu/libgcrypt.so.11.8.3)
==29749==    by 0x617C077: gcry_mpi_print (in
/lib/i386-linux-gnu/libgcrypt.so.11.8.3)
==29749==    by 0x5F26E65: ??? (in
/usr/lib/i386-linux-gnu/libgnutls.so.26.22.6)
==29749==    by 0x5ED152C: ??? (in
/usr/lib/i386-linux-gnu/libgnutls.so.26.22.6)
==29749==    by 0x5ED180D: ??? (in
/usr/lib/i386-linux-gnu/libgnutls.so.26.22.6)
==29749==    by 0x5EB4B25: ??? (in
/usr/lib/i386-linux-gnu/libgnutls.so.26.22.6)
==29749==    by 0x5EB14D3: ??? (in
/usr/lib/i386-linux-gnu/libgnutls.so.26.22.6)
==29749==    by 0x5EB1EF0: gnutls_handshake (in
/usr/lib/i386-linux-gnu/libgnutls.so.26.22.6)
==29749==    by 0x5D30F26: schan_imp_handshake (schannel_gnutls.c:200)
==29749==    by 0x5D2F654: schan_InitializeSecurityContextW (schannel.c:893)
==29749==    by 0x5D3616C: InitializeSecurityContextW (wrapper.c:346)
==29749==    by 0x4E95BAE: netconn_secure_connect (net.c:478)
==29749==    by 0x4E9EA75: open_connection (request.c:1064)
==29749==    by 0x4E9FBED: send_request (request.c:1185)
==29749==    by 0x4EA124C: WinHttpSendRequest (request.c:1272)
==29749==    by 0x4D68D6A: test_secure_connection (winhttp.c:858)
==29749==    by 0x4D68D6A: func_winhttp (???:0)
==29749==    by 0x4D57F28: main (test.h:584)
==29749==  Uninitialised value was created by a stack allocation
==29749==    at 0x5ED14AF: ??? (in
/usr/lib/i386-linux-gnu/libgnutls.so.26.22.6)

[Bug 36599] winhttp/tests/winhttp.c: test_secure_connection() fails under valgrind

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

--- Comment #3 from Hans Leidekker  ---
Tests pass here under valgrind.

[Bug 36599] winhttp/tests/winhttp.c: test_secure_connection() fails under valgrind

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #4 from Austin English  ---
(In reply to Hans Leidekker from comment #3)
> Tests pass here under valgrind.

Here as well in wine-2.3-135-gfaa6ff5

[Bug 36599] winhttp/tests/winhttp.c: test_secure_connection() fails under valgrind

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

Alexandre Julliard  changed:

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

--- Comment #5 from Alexandre Julliard  ---
Closing bugs fixed in 2.4.