[Bug 36341] New: valgrind shows some possible leaks under crypt32/tests/base64.c

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

Bug ID: 36341
           Summary: valgrind shows some possible leaks under
                    crypt32/tests/base64.c
           Product: Wine
           Version: 1.7.17
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: normal
          Priority: P2
         Component: crypt32
          Assignee: [e-mail]
          Reporter: [e-mail]

==21036== 2,628 bytes in 73 blocks are possibly lost in loss record 250 of 266
==21036==    at 0x7BC50A9B: RtlAllocateHeap (heap.c:255)
==21036==    by 0x4F815DF: CryptMemAlloc (main.c:124)
==21036==    by 0x4F92C4B: crypt_oid_init (oid.c:1403)
==21036==    by 0x4F811F8: DllMain (main.c:46)
==21036==    by 0x4FA5F36: __wine_spec_dll_entry (dll_entry.c:40)
==21036==    by 0x7BC53B24: ??? (in
/home/austin/wine-valgrind-known/dlls/ntdll/ntdll.dll.so)
==21036==    by 0x7BC5644E: MODULE_InitDLL (loader.c:1068)
==21036==    by 0x7BC56AE7: process_attach (loader.c:1157)
==21036==    by 0x7BC56AB9: process_attach (loader.c:1149)
==21036==    by 0x7BC56CEF: attach_process_dlls (loader.c:2769)
==21036==    by 0x401BC7C: ??? (in
/home/austin/wine-valgrind-known/libs/wine/libwine.so.1.0)
==21036==

[Bug 36341] valgrind shows some possible leaks under crypt32/tests/base64.c

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

Austin English  changed:

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

[Bug 36341] valgrind shows some possible leaks under crypt32/tests/base64.c

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

--- Comment #1 from Bruno Jesus  ---
The memory is released in the DLL_PROCESS_DETACH. Why would it not be called?

[Bug 36341] valgrind shows some possible leaks under crypt32/tests/ base64.c

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

[e-mail] changed:

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

--- Comment #2 from [e-mail] ---
Hi Austin,
You might want to re-test this, I am not reported any leaks in `base64` tests
with 1.9.22 and wineserver running in a separate process

[Bug 36341] valgrind shows some possible leaks under crypt32/tests/ base64.c

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

--- Comment #3 from [e-mail] ---
Please read "wineserver launched beforehand and keep running to check leaks in
tests only" in my previous comment where "separate process", sorry for typo

[Bug 36341] valgrind shows some possible leaks under crypt32/tests/ base64.c

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

Austin English  changed:

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

--- Comment #4 from Austin English  ---
(In reply to donanykey from comment #2)
> Hi Austin,
> You might want to re-test this, I am not reported any leaks in `base64`
> tests with 1.9.22 and wineserver running in a separate process

I can't reproduce either in wine 2.1.

[Bug 36341] valgrind shows some possible leaks under crypt32/tests/ base64.c

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

Alexandre Julliard  changed:

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

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