[Bug 36606] New: valgrind shows uninitialized memory use in ws2_32/tests/protocol.c
https://bugs.winehq.org/show_bug.cgi?id=36606 Bug ID: 36606 Summary: valgrind shows uninitialized memory use in ws2_32/tests/protocol.c Product: Wine Version: 1.7.19 Hardware: x86 OS: Linux Status: NEW Keywords: download, source, testcase Severity: normal Priority: P2 Component: winsock Assignee: wine-bugs(a)winehq.org Reporter: austinenglish(a)gmail.com ==21539== Invalid write of size 4 ==21539== at 0x402F31D: memset (vg_replace_strmem.c:1087) ==21539== by 0x4E1CFD5: WS_EnterSingleProtocolA (socket.c:1778) ==21539== by 0x4E1D22C: WS_EnumProtocols (socket.c:1832) ==21539== by 0x4E2CC95: WSAEnumProtocolsA (socket.c:7483) ==21539== by 0x4CA2650: test_WSAEnumProtocolsA (protocol.c:110) ==21539== by 0x4CA2D46: func_protocol (protocol.c:202) ==21539== by 0x4CCAD54: run_test (test.h:584) ==21539== by 0x4CCB143: main (test.h:654) ==21539== Address 0x499bf18 is 0 bytes after a block of size 744 alloc'd ==21539== at 0x7BC4C6DF: notify_alloc (heap.c:255) ==21539== by 0x7BC50F23: RtlAllocateHeap (heap.c:1716) ==21539== by 0x4CA2624: test_WSAEnumProtocolsA (protocol.c:106) ==21539== by 0x4CA2D46: func_protocol (protocol.c:202) ==21539== by 0x4CCAD54: run_test (test.h:584) ==21539== by 0x4CCB143: main (test.h:654) ==21539== -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=36606 --- Comment #1 from Bruno Jesus <00cpxxx(a)gmail.com> --- I can't find this issue... The buffer is 744 bytes long and the memset writes 744 bytes, I don't see this invalid write anywhere =/ -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=36606 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |valgrind -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=36606 --- Comment #2 from Austin English <austinenglish(a)gmail.com> --- austin(a)aw25 ~/wine-valgrind/dlls/ws2_32/tests $ make protocol.ok gcc -m32 -c -o sock.o sock.c -I. -I../../../include -D__WINESRC__ -D_REENTRANT -fPIC -Wall -pipe \ -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers \ -Wstrict-prototypes -Wtype-limits -Wunused-but-set-parameter -Wvla -Wwrite-strings -Wpointer-arith \ -Wlogical-op -gdwarf-2 -gstrict-dwarf -fno-omit-frame-pointer -g -O0 -fno-inline ../../../tools/winegcc/winegcc -o ws2_32_test.exe.so -B../../../tools/winebuild -m32 -fasynchronous-unwind-tables \ protocol.o sock.o testlist.o -lws2_32 -luser32 -lm ../../../tools/runtest -q -P wine -T ../../.. -M ws2_32.dll -p ws2_32_test.exe.so protocol && touch protocol.ok preloader: Warning: failed to reserve range 00110000-68000000 preloader: Warning: failed to reserve range 7f000000-82000000 fixme:winsock:WS_EnterSingleProtocolW unknown Protocol <0x0000dead> fixme:winsock:WS_EnterSingleProtocolW unknown Protocol <0x0000cafe> ==18265== Invalid write of size 4 ==18265== at 0x402E46B: memset (mc_replace_strmem.c:1055) ==18265== by 0x50A8FD5: WS_EnterSingleProtocolA (socket.c:1778) ==18265== by 0x50A922C: WS_EnumProtocols (socket.c:1832) ==18265== by 0x50B8C95: WSAEnumProtocolsA (socket.c:7483) ==18265== by 0x4F596A8: test_WSAEnumProtocolsA (protocol.c:110) ==18265== by 0x4F59D9E: func_protocol (protocol.c:202) ==18265== by 0x4F80BB1: run_test (test.h:584) ==18265== by 0x4F80FA0: main (test.h:654) ==18265== Address 0x4a9d5c0 is 0 bytes after a block of size 744 alloc'd ==18265== at 0x7BC4C6EB: notify_alloc (heap.c:255) ==18265== by 0x7BC50F2F: RtlAllocateHeap (heap.c:1716) ==18265== by 0x4F5967C: test_WSAEnumProtocolsA (protocol.c:106) ==18265== by 0x4F59D9E: func_protocol (protocol.c:202) ==18265== by 0x4F80BB1: run_test (test.h:584) ==18265== by 0x4F80FA0: main (test.h:654) ==18265== austin(a)aw25 ~/wine-valgrind/dlls/ws2_32/tests $ uname -a Linux aw25 3.13.3-gentoo-devlight-1 #1 SMP Thu Feb 20 00:10:55 Local time zone must be set--see zic x86_64 Intel(R) Core(TM) i7 CPU 960 @ 3.20GHz GenuineIntel GNU/Linux is there some logs you'd like? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=36606 --- Comment #3 from Bruno Jesus <00cpxxx(a)gmail.com> --- (In reply to Austin English from comment #2)
is there some logs you'd like?
No, thanks. I think I've found it using a lot of printf. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=36606 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |00cpxxx(a)gmail.com --- Comment #4 from Bruno Jesus <00cpxxx(a)gmail.com> --- Created attachment 48790 --> https://bugs.winehq.org/attachment.cgi?id=48790 patch Please try the attached patch. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=36606 --- Comment #5 from Austin English <austinenglish(a)gmail.com> --- (In reply to Bruno Jesus from comment #4)
Created attachment 48790 [details] patch
Please try the attached patch.
Works for me, thanks. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=36606 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |6f47a9d9d83ff53d486cf3c163f | |eb7df7f4bfcd0 Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #6 from Bruno Jesus <00cpxxx(a)gmail.com> --- Patch commited. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=36606 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #7 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 1.7.21. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
wine-bugs@winehq.org