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

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

Bug ID: 36431
           Summary: valgrind shows a possible leak in
                    shell32/tests/ebrowser.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]

==28261== 28 bytes in 1 blocks are possibly lost in loss record 279 of 995
==28261==    at 0x7BC50A9B: RtlAllocateHeap (heap.c:255)
==28261==    by 0x5C44BB8: apartment_getclassobject (compobj.c:526)
==28261==    by 0x5C47FC6: get_inproc_class_object (compobj.c:2894)
==28261==    by 0x5C4AD2F: CoGetClassObject (compobj.c:3032)
==28261==    by 0x5C4B4B7: CoCreateInstance (compobj.c:3197)
==28261==    by 0x4D82686: func_ebrowser (ebrowser.c:94)
==28261==    by 0x4DC8888: run_test (test.h:584)
==28261==    by 0x4DC8C77: main (test.h:654)
==28261==

[Bug 36431] valgrind shows a possible leak in shell32/tests/ebrowser.c

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

Austin English  changed:

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

[Bug 36431] valgrind shows a possible leak in shell32/tests/ebrowser.c

[WineHQ Bugzilla]

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

joaopa  changed:

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

--- Comment #1 from joaopa  ---
Does the bug still occur with wine-4.19?

[Bug 36431] valgrind shows a possible leak in shell32/tests/ebrowser.c

[WineHQ Bugzilla]

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

Zebediah Figura  changed:

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

--- Comment #2 from Zebediah Figura  ---
(In reply to joaopa from comment #1)
> Does the bug still occur with wine-4.19?

I suspect you could test this without significant difficulty.

[Bug 36431] valgrind shows a possible leak in shell32/tests/ebrowser.c

[WineHQ Bugzilla]

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

--- Comment #3 from joaopa  ---
> Does the bug still occur with wine-4.19?

I do not know how to use valgrind.

[Bug 36431] valgrind shows a possible leak in shell32/tests/ebrowser.c

[WineHQ Bugzilla]

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

--- Comment #4 from Zebediah Figura  ---
(In reply to joaopa from comment #3)
> > Does the bug still occur with wine-4.19?
>
> I do not know how to use valgrind.

Quite simple: `valgrind --trace-children=yes --leak-check=full wine
shell32_test.exe ebrowser`

[Bug 36431] valgrind shows a possible leak in shell32/tests/ebrowser.c

[WineHQ Bugzilla]

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

--- Comment #5 from joaopa  ---
Created attachment 65587
  --> https://bugs.winehq.org/attachment.cgi?id=65587
console output from valgrind

I obtained this console output. I did not see
> by 0x4D82686: func_ebrowser (ebrowser.c:94)

Does that mean that the bug is fixed?

[Bug 36431] valgrind shows a possible leak in shell32/tests/ebrowser.c

[WineHQ Bugzilla]

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

Zebediah Figura  changed:

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

--- Comment #6 from Zebediah Figura  ---
(In reply to joaopa from comment #5)
> Created attachment 65587 [details]
> console output from valgrind
>
> I obtained this console output. I did not see
> > by 0x4D82686: func_ebrowser (ebrowser.c:94)
>
> Does that mean that the bug is fixed?

It would seem so.

[Bug 36431] valgrind shows a possible leak in shell32/tests/ebrowser.c

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

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

--- Comment #7 from Alexandre Julliard  ---
Closing bugs fixed in 4.20.