[Bug 36430] New: valgrind shows a possible leak in shell32/tests/autocomplete.c

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

Bug ID: 36430
           Summary: valgrind shows a possible leak in
                    shell32/tests/autocomplete.c
           Product: Wine
           Version: 1.7.18
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: normal
          Priority: P2
         Component: shell32
          Assignee: [e-mail]
          Reporter: [e-mail]

==7141== 28 bytes in 1 blocks are possibly lost in loss record 207 of 679
==7141==    at 0x7BC50A9B: RtlAllocateHeap (heap.c:255)
==7141==    by 0x5A28BB8: apartment_getclassobject (compobj.c:526)
==7141==    by 0x5A2BFC6: get_inproc_class_object (compobj.c:2894)
==7141==    by 0x5A2ED2F: CoGetClassObject (compobj.c:3032)
==7141==    by 0x5A2F4B7: CoCreateInstance (compobj.c:3197)
==7141==    by 0x4B5C18E: func_autocomplete (autocomplete.c:44)
==7141==    by 0x4BA8888: run_test (test.h:584)
==7141==    by 0x4BA8C77: main (test.h:654)
==7141==

[Bug 36430] valgrind shows a possible leak in shell32/tests/autocomplete.c

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

Austin English  changed:

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

[Bug 36430] valgrind shows a possible leak in shell32/tests/autocomplete.c

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

Stefan Dösinger  changed:

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

--- Comment #1 from Stefan Dösinger  ---
Mark Jansen reported a memory leak problem in autocomplete.c on IRC:

(12:02:14) Mark_Jansen: hello!
(12:02:20) Mark_Jansen: there is a small memory leak in shell32/autocomplete.c
(12:02:26) Mark_Jansen:
https://source.winehq.org/git/wine.git/blob/709935314458bd0ce27aab3986ae98cc...
(12:02:32) Mark_Jansen:  hr = IEnumString_Next(This->enumstr, 1, &strs,
&fetched);
(12:02:40) Mark_Jansen: this 'strs' should be freed after using it

I can't say for sure if this is the same problem Valgrind found here.

[Bug 36430] valgrind shows a possible leak in shell32/tests/autocomplete.c

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

Gabriel Ivăncescu  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED
                 CC|                            |[e-mail]

--- Comment #2 from Gabriel Ivăncescu  ---
This has been fixed for a while as the autocomplete enumeration was revamped,
but possibly even way before that (so I don't know the SHA1 that did it).

[Bug 36430] valgrind shows a possible leak in shell32/tests/autocomplete.c

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

Alexandre Julliard  changed:

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

--- Comment #3 from Alexandre Julliard  ---
Closing bugs fixed in 4.0-rc2.