[Bug 36637] New: wininet/tests/http.c crashes under valgrind

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

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

Created attachment 48678
  --> https://bugs.winehq.org/attachment.cgi?id=48678
valgrind log/backtrace

See also bug 36004 for a crash with warn+heap.

wine: Unhandled page fault on read access to 0x04760000 at address 0x7bc4d939
(thread 0033), starting debugger...
Backtrace:
=>0 0x7bc4d939 HEAP_CreateFreeBlock+0xb5(subheap=0x46f0014, ptr=0x4755cb8,
size=0xa348) [/home/austin/wine-valgrind/dlls/ntdll/heap.c:591] in ntdll
(0x04d0eaa4)
  1 0x7bc4dc61 HEAP_ShrinkBlock+0x42(subheap=0x46f0014, pArena=0x4755c88,
size=0x28) [/home/austin/wine-valgrind/dlls/ntdll/heap.c:694] in ntdll
(0x04d0eac8)
  2 0x7bc50f6e RtlAllocateHeap+0x2fb(heap=<couldn't compute location>,
flags=<couldn't compute location>, size=<couldn't compute location>)
[/home/austin/wine-valgrind/dlls/ntdll/heap.c:1713] in ntdll (0x04d0eb58)
  3 0x04d35001 heap_alloc+0x2b(len=0x20)
[/home/austin/wine-valgrind/dlls/wininet/internet.h:116] in wininet
(0x04d0eb98)
  4 0x04d3f383 HttpQueryInfoA+0x13a(hHttpRequest=<couldn't compute location>,
dwInfoLevel=<couldn't compute location>, lpBuffer=<couldn't compute location>,
lpdwBufferLength=<couldn't compute location>, lpdwIndex=<couldn't compute
location>) [/home/austin/wine-valgrind/dlls/wininet/http.c:3934] in wininet
(0x04d0ebf8)
  5 0x0495e5b8 InternetReadFile_test+0x16ef(flags=0, test=0x49a4780)
[/home/austin/wine-valgrind/dlls/wininet/tests/http.c:639] in wininet_test
(0x04d0fc98)

I'll attach the full log here and file separate bugs for the leaks/etc.

[Bug 36637] wininet/tests/http.c crashes under valgrind

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |36638

[Bug 36637] wininet/tests/http.c crashes under valgrind

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |36639

[Bug 36637] wininet/tests/http.c crashes under valgrind

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |36641

[Bug 36637] wininet/tests/http.c crashes under valgrind

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

Austin English  changed:

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

[Bug 36637] wininet/tests/http.c crashes under valgrind

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

Jacek Caban  changed:

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

--- Comment #1 from Jacek Caban  ---
Does it still crash? I can't reproduce it here.

[Bug 36637] wininet/tests/http.c crashes under valgrind

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

--- Comment #2 from Austin English  ---
(In reply to Jacek Caban from comment #1)
> Does it still crash? I can't reproduce it here.

In 1.7.38, I get a hang. I filed bug 38217 for that.

[Bug 36637] wininet/tests/http.c crashes under valgrind

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

Austin English  changed:

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

--- Comment #3 from Austin English  ---
No longer crashes or hangs in wine-1.7.51-225-g3966aff

[Bug 36637] wininet/tests/http.c crashes under valgrind

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

Alexandre Julliard  changed:

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

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