From 2a1fd9851c33fae7d8cc3b6ef63e9bf8a0effe4f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9mi=20Bernon?= <rbernon@codeweavers.com>
Date: Fri, 22 Sep 2023 20:56:17 +0200
Subject: [PATCH] dmusic: Avoid leaking articulations when freeing regions.

---
 dlls/dmusic/instrument.c | 24 +++++++++++++++---------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/dlls/dmusic/instrument.c b/dlls/dmusic/instrument.c
index 8ce3c0bc979..54ce7508ee4 100644
--- a/dlls/dmusic/instrument.c
+++ b/dlls/dmusic/instrument.c
@@ -44,6 +44,19 @@ struct region
     BOOL loop_present;
 };
 
+static void region_destroy(struct region *region)
+{
+    struct articulation *articulation, *next;
+
+    LIST_FOR_EACH_ENTRY_SAFE(articulation, next, &region->articulations, struct articulation, entry)
+    {
+        list_remove(&articulation->entry);
+        free(articulation);
+    }
+
+    free(region);
+}
+
 struct instrument
 {
     IDirectMusicInstrument IDirectMusicInstrument_iface;
@@ -122,15 +135,8 @@ static ULONG WINAPI instrument_Release(LPDIRECTMUSICINSTRUMENT iface)
 
         LIST_FOR_EACH_ENTRY_SAFE(region, next_region, &This->regions, struct region, entry)
         {
-            LIST_FOR_EACH_ENTRY_SAFE(articulation, next_articulation, &region->articulations,
-                    struct articulation, entry)
-            {
-                list_remove(&articulation->entry);
-                free(articulation);
-            }
-
             list_remove(&region->entry);
-            free(region);
+            region_destroy(region);
         }
 
         collection_internal_release(This->collection);
@@ -322,7 +328,7 @@ static HRESULT parse_rgn_chunk(struct instrument *This, IStream *stream, struct
         if (FAILED(hr)) break;
     }
 
-    if (FAILED(hr)) free(region);
+    if (FAILED(hr)) region_destroy(region);
     else list_add_tail(&This->regions, &region->entry);
 
     return hr;