Alistair Leslie-Hughes : dmime: Free list when SegTrigger track is destroyed.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 71b433bfed082486956eebd901f13ff9c42bd5dc
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=71b433bfed082486956eebd90...

Author: Alistair Leslie-Hughes 
Date:   Thu Nov 14 20:40:27 2019 +0100

dmime: Free list when SegTrigger track is destroyed.

Signed-off-by: Alistair Leslie-Hughes 
Signed-off-by: Michael Stefaniuc 
Signed-off-by: Alexandre Julliard

---

dlls/dmime/segtriggertrack.c | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/dlls/dmime/segtriggertrack.c b/dlls/dmime/segtriggertrack.c
index 4b3a71c9a4..3050a5031b 100644
--- a/dlls/dmime/segtriggertrack.c
+++ b/dlls/dmime/segtriggertrack.c
@@ -21,6 +21,8 @@
 #include "dmime_private.h"
 #include "dmobject.h"

+#include "wine/heap.h"
+
 WINE_DEFAULT_DEBUG_CHANNEL(dmime);
 WINE_DECLARE_DEBUG_CHANNEL(dmfile);

@@ -81,7 +83,19 @@ static ULONG WINAPI segment_track_Release(IDirectMusicTrack8 *iface)
     TRACE("(%p) ref=%d\n", This, ref);

if (!ref) {
-        HeapFree(GetProcessHeap(), 0, This);
+        struct list *cursor, *cursor2;
+        DMUS_PRIVATE_SEGMENT_ITEM *item;
+
+        LIST_FOR_EACH_SAFE(cursor, cursor2, &This->Items) {
+            item = LIST_ENTRY(cursor, DMUS_PRIVATE_SEGMENT_ITEM, entry);
+            list_remove(cursor);
+
+            if (item->pObject)
+                IDirectMusicObject_Release(item->pObject);
+            heap_free(item);
+        }
+
+        heap_free(This);
         DMIME_UnlockModule();
     }