http://bugs.winehq.org/show_bug.cgi?id=26042
Summary: advapi32/crypt tests show two leaks under valgrind Product: Wine Version: 1.3.13 Platform: x86 OS/Version: Linux Status: NEW Keywords: download, source, testcase Severity: minor Priority: P2 Component: advapi32 AssignedTo: wine-bugs@winehq.org ReportedBy: austinenglish@gmail.com
12 bytes in 1 blocks are definitely lost at notify_alloc (heap.c:254) by RtlAllocateHeap (heap.c:1701) by HeapAlloc (heap.c:267) by GlobalAlloc (heap.c:360) by LocalAlloc (heap.c:960) by CryptGenKey (crypt.c:1433) by test_incorrect_api_usage (crypt.c:241) by func_crypt (crypt.c:1130) by run_test (test.h:556) by main (test.h:624) ... 996 bytes in 1 blocks are definitely lost at notify_alloc (heap.c:254) by RtlAllocateHeap (heap.c:1701) by ??? by ??? by ??? by CryptGenKey (crypt.c:1441) by test_incorrect_api_usage (crypt.c:241) by func_crypt (crypt.c:1130) by run_test (test.h:556) by main (test.h:624)
http://bugs.winehq.org/show_bug.cgi?id=26042
--- Comment #1 from Nikolay Sivov bunglehead@gmail.com 2011-02-09 19:35:05 CST --- This is what's this test is about actually. Context is released before releasing key, and key release call supposed to fail after that as it does now.
Probably we should track all keys and free them while releasing a provider, I'm not sure it's important to implement.
http://bugs.winehq.org/show_bug.cgi?id=26042
--- Comment #2 from butraxz@gmail.com 2013-08-05 12:47:32 CDT --- This ticket has not been updated for over 900 days. Development recommends to check the status on your bug every release or two and let to be known if the bug is still present. If not, mark it fixed.
Is this still an issue with 1.7.0 or higher ?
http://bugs.winehq.org/show_bug.cgi?id=26042
--- Comment #3 from Austin English austinenglish@gmail.com 2013-08-05 18:28:15 CDT --- Still in 1.7.0.
https://bugs.winehq.org/show_bug.cgi?id=26042
--- Comment #4 from Austin English austinenglish@gmail.com --- ==7987== 12 bytes in 1 blocks are definitely lost in loss record 16 of 241 ==7987== at 0x7BC4C6B7: notify_alloc (heap.c:255) ==7987== by 0x7BC50EFB: RtlAllocateHeap (heap.c:1716) ==7987== by 0x7B8457FF: HeapAlloc (heap.c:271) ==7987== by 0x7B845A26: GlobalAlloc (heap.c:364) ==7987== by 0x7B846E63: LocalAlloc (heap.c:967) ==7987== by 0x54587ED: CryptGenKey (crypt.c:1443) ==7987== by 0x4E679D4: test_incorrect_api_usage (crypt.c:254) ==7987== by 0x4E6C061: func_crypt (crypt.c:1183) ==7987== by 0x4EB6E10: run_test (test.h:584) ==7987== by 0x4EB71FF: main (test.h:654) ==7987== .. ==7987== 996 bytes in 1 blocks are possibly lost in loss record 220 of 241 ==7987== at 0x7BC4C6B7: notify_alloc (heap.c:255) ==7987== by 0x7BC50EFB: RtlAllocateHeap (heap.c:1716) ==7987== by 0x60A1770: ??? ==7987== by 0x60AC6FE: ??? ==7987== by 0x60B142A: ??? ==7987== by 0x545884E: CryptGenKey (crypt.c:1451) ==7987== by 0x4E679D4: test_incorrect_api_usage (crypt.c:254) ==7987== by 0x4E6C061: func_crypt (crypt.c:1183) ==7987== by 0x4EB6E10: run_test (test.h:584) ==7987== by 0x4EB71FF: main (test.h:654) ==7987==
still in wine-1.7.17-42-g24c5728
https://bugs.winehq.org/show_bug.cgi?id=26042
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |valgrind
https://bugs.winehq.org/show_bug.cgi?id=26042
Zhiyi Zhang zzhang@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |zzhang@codeweavers.com
--- Comment #5 from Zhiyi Zhang zzhang@codeweavers.com --- $ valgrind --leak-check=full ../../../wine advapi32_test.exe.so crypt ==22872== Memcheck, a memory error detector ==22872== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al. ==22872== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info ==22872== Command: ../../../wine advapi32_test.exe.so crypt ==22872== ==22875== ==22875== HEAP SUMMARY: ==22875== in use at exit: 8,977 bytes in 42 blocks ==22875== total heap usage: 69 allocs, 27 frees, 13,990 bytes allocated ==22875== ==22875== LEAK SUMMARY: ==22875== definitely lost: 0 bytes in 0 blocks ==22875== indirectly lost: 0 bytes in 0 blocks ==22875== possibly lost: 0 bytes in 0 blocks ==22875== still reachable: 8,977 bytes in 42 blocks ==22875== suppressed: 0 bytes in 0 blocks ==22875== Reachable blocks (those to which a pointer was found) are not shown. ==22875== To see them, rerun with: --leak-check=full --show-leak-kinds=all ==22875== ==22875== For counts of detected and suppressed errors, rerun with: -v ==22875== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0) 0008:crypt: 109 tests executed (0 marked as todo, 0 failures), 0 skipped.
$ ../../../wine --version wine-3.3-1-g66dae6b1f2
https://bugs.winehq.org/show_bug.cgi?id=26042
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED
--- Comment #6 from Austin English austinenglish@gmail.com --- I get a different leak in 4.0-rc2: ==24892== 416 bytes in 1 blocks are definitely lost in loss record 290 of 383 ==24892== at 0x7BC483AD: notify_alloc (heap.c:260) ==24892== by 0x7BC4B8D1: RtlAllocateHeap (heap.c:1726) ==24892== by 0x5BBEB7D: ??? ==24892== by 0x5BC12A8: ??? ==24892== by 0x5A76E5E: ??? ==24892== by 0x5A809C9: ??? ==24892== by 0x5A81A70: ??? ==24892== by 0x5003F30: CryptCreateHash (crypt.c:765) ==24892== by 0x4C1FE4C: test_verify_sig (crypt.c:436) ==24892== by 0x4C21EF8: func_crypt (crypt.c:1193) ==24892== by 0x4C6AC1E: run_test (test.h:617) ==24892== by 0x4C6B65C: main (test.h:701) ==24892==
marking fixed.
https://bugs.winehq.org/show_bug.cgi?id=26042
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #7 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 4.0-rc3.
-
wine-bugs＠winehq.org