[Bug 43310] New: valgrind shows several definite leaks in dlls/advapi32/ tests/security.c

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

Bug ID: 43310
           Summary: valgrind shows several definite leaks in
                    dlls/advapi32/tests/security.c
           Product: Wine
           Version: 2.12
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase, valgrind
          Severity: minor
          Priority: P2
         Component: advapi32
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

==25974== 13 bytes in 1 blocks are definitely lost in loss record 22 of 264
==25974==    at 0x7BC50812: notify_alloc (heap.c:254)
==25974==    by 0x7BC54C93: RtlAllocateHeap (heap.c:1716)
==25974==    by 0x7B449A29: HeapAlloc (heap.c:271)
==25974==    by 0x7B44B158: LocalAlloc (heap.c:970)
==25974==    by 0x4E656F5: ConvertSidToStringSidA (security.c:5283)
==25974==    by 0x4A68048: test_token_label (security.c:6787)
==25974==    by 0x4A69DD5: func_security (security.c:7131)
==25974==    by 0x4A7926E: run_test (test.h:603)
==25974==    by 0x4A796C0: main (test.h:687)
==25974== 
==25974== 20 bytes in 1 blocks are definitely lost in loss record 54 of 264
==25974==    at 0x7BC50812: notify_alloc (heap.c:254)
==25974==    by 0x7BC54C93: RtlAllocateHeap (heap.c:1716)
==25974==    by 0x7B449A29: HeapAlloc (heap.c:271)
==25974==    by 0x7B44B158: LocalAlloc (heap.c:970)
==25974==    by 0x4E60D26: GetSecurityInfo (security.c:3267)
==25974==    by 0x4A66125: test_system_security_access (security.c:6479)
==25974==    by 0x4A69DC1: func_security (security.c:7127)
==25974==    by 0x4A7926E: run_test (test.h:603)
==25974==    by 0x4A796C0: main (test.h:687)
==25974== 
==25974== 148 bytes in 1 blocks are definitely lost in loss record 183 of 264
==25974==    at 0x7BC50812: notify_alloc (heap.c:254)
==25974==    by 0x7BC54C93: RtlAllocateHeap (heap.c:1716)
==25974==    by 0x7B449A29: HeapAlloc (heap.c:271)
==25974==    by 0x7B44B158: LocalAlloc (heap.c:970)
==25974==    by 0x4E60D26: GetSecurityInfo (security.c:3267)
==25974==    by 0x4A5BCC5: test_GetSecurityInfo (security.c:4616)
==25974==    by 0x4A69D85: func_security (security.c:7115)
==25974==    by 0x4A7926E: run_test (test.h:603)
==25974==    by 0x4A796C0: main (test.h:687)
==25974== 
==25950== 13 bytes in 1 blocks are definitely lost in loss record 21 of 258
==25950==    at 0x7BC50812: notify_alloc (heap.c:254)
==25950==    by 0x7BC54C93: RtlAllocateHeap (heap.c:1716)
==25950==    by 0x7B449A29: HeapAlloc (heap.c:271)
==25950==    by 0x7B44B158: LocalAlloc (heap.c:970)
==25950==    by 0x4E656F5: ConvertSidToStringSidA (security.c:5283)
==25950==    by 0x4A68048: test_token_label (security.c:6787)
==25950==    by 0x4A69DD5: func_security (security.c:7131)
==25950==    by 0x4A7926E: run_test (test.h:603)
==25950==    by 0x4A796C0: main (test.h:687)
==25950== 
==25950== 20 bytes in 1 blocks are definitely lost in loss record 52 of 258
==25950==    at 0x7BC50812: notify_alloc (heap.c:254)
==25950==    by 0x7BC54C93: RtlAllocateHeap (heap.c:1716)
==25950==    by 0x7B449A29: HeapAlloc (heap.c:271)
==25950==    by 0x7B44B158: LocalAlloc (heap.c:970)
==25950==    by 0x4E60D26: GetSecurityInfo (security.c:3267)
==25950==    by 0x4A66125: test_system_security_access (security.c:6479)
==25950==    by 0x4A69DC1: func_security (security.c:7127)
==25950==    by 0x4A7926E: run_test (test.h:603)
==25950==    by 0x4A796C0: main (test.h:687)
==25950== 
==25950== 84 bytes in 1 blocks are definitely lost in loss record 153 of 258
==25950==    at 0x7BC50812: notify_alloc (heap.c:254)
==25950==    by 0x7BC54C93: RtlAllocateHeap (heap.c:1716)
==25950==    by 0x4A68DF7: test_token_security_descriptor (security.c:6921)
==25950==    by 0x4A69DDA: func_security (security.c:7134)
==25950==    by 0x4A7926E: run_test (test.h:603)
==25950==    by 0x4A796C0: main (test.h:687)
==25950== 
==25950== 148 bytes in 1 blocks are definitely lost in loss record 179 of 258
==25950==    at 0x7BC50812: notify_alloc (heap.c:254)
==25950==    by 0x7BC54C93: RtlAllocateHeap (heap.c:1716)
==25950==    by 0x7B449A29: HeapAlloc (heap.c:271)
==25950==    by 0x7B44B158: LocalAlloc (heap.c:970)
==25950==    by 0x4E60D26: GetSecurityInfo (security.c:3267)
==25950==    by 0x4A5BCC5: test_GetSecurityInfo (security.c:4616)
==25950==    by 0x4A69D85: func_security (security.c:7115)
==25950==    by 0x4A7926E: run_test (test.h:603)
==25950==    by 0x4A796C0: main (test.h:687)
==25950==

[Bug 43310] valgrind shows several definite leaks in dlls/advapi32/tests/ security.c

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

Matteo Bruni  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
      Fixed by SHA1|                            |9ccce0ce645f089063ed61bec53
                   |                            |644a138ac0a37
         Resolution|---                         |FIXED

--- Comment #1 from Matteo Bruni  ---
This should be fixed by 9ccce0ce645f089063ed61bec53644a138ac0a37, aside from
the last block which is expected:

/* If we don't ask for the security descriptor, Windows will still give us
   the other stuff, leaving us no way to free it.  */

[Bug 43310] valgrind shows several definite leaks in dlls/advapi32/tests/ security.c

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

Alexandre Julliard  changed:

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

--- Comment #2 from Alexandre Julliard  ---
Closing bugs fixed in 3.0-rc6.