matroska: Use g_array_unref everywhere
authorRafał Dzięgiel <rafostar.github@gmail.com>
Fri, 12 Nov 2021 10:28:23 +0000 (11:28 +0100)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Fri, 12 Nov 2021 12:28:40 +0000 (12:28 +0000)
Instead of using g_array_free which is not thread safe use g_array_unref instead

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/1203>

subprojects/gst-plugins-good/gst/matroska/ebml-read.c
subprojects/gst-plugins-good/gst/matroska/matroska-demux.c
subprojects/gst-plugins-good/gst/matroska/matroska-ids.c
subprojects/gst-plugins-good/gst/matroska/matroska-read-common.c

index f66990b35e17b814d68fbbcfd01f7f70fef3bcce..6ffc9f11138c5df2579dd5f6e38b8cb23e2c901a 100644 (file)
@@ -175,7 +175,7 @@ void
 gst_ebml_read_clear (GstEbmlRead * ebml)
 {
   if (ebml->readers)
-    g_array_free (ebml->readers, TRUE);
+    g_array_unref (ebml->readers);
   ebml->readers = NULL;
   if (ebml->buf) {
     gst_buffer_unmap (ebml->buf, &ebml->map);
index 1eb429f9382b868d7054cfbd955be26eb8854f7b..ae3588487b586fdf97e885a5e52d8cfe9c81ad04 100644 (file)
@@ -328,7 +328,7 @@ gst_matroska_demux_reset (GstElement * element)
   demux->tracks_parsed = FALSE;
 
   if (demux->clusters) {
-    g_array_free (demux->clusters, TRUE);
+    g_array_unref (demux->clusters);
     demux->clusters = NULL;
   }
 
index eca5279224fc63a54f2ad5a42d30fdbf38506053..f11b7c2ce31f7e5296b93c19dc8cb96042672330 100644 (file)
@@ -349,14 +349,14 @@ gst_matroska_track_free (GstMatroskaTrackContext * track)
 
       g_free (enc->comp_settings);
     }
-    g_array_free (track->encodings, TRUE);
+    g_array_unref (track->encodings);
   }
 
   if (track->tags)
     gst_tag_list_unref (track->tags);
 
   if (track->index_table)
-    g_array_free (track->index_table, TRUE);
+    g_array_unref (track->index_table);
 
   if (track->stream_headers)
     gst_buffer_list_unref (track->stream_headers);
index e2bedc8c1ae1218f690ca79d54ec713dcd0958b3..2b63c03d56e76b0ff6753c6216884d9f48c18413 100644 (file)
@@ -1809,7 +1809,7 @@ gst_matroska_read_common_parse_index (GstMatroskaReadCommon * common,
   guint i;
 
   if (common->index)
-    g_array_free (common->index, TRUE);
+    g_array_unref (common->index);
   common->index =
       g_array_sized_new (FALSE, FALSE, sizeof (GstMatroskaIndex), 128);
 
@@ -1897,7 +1897,7 @@ gst_matroska_read_common_parse_index (GstMatroskaReadCommon * common,
 
   /* sanity check; empty index normalizes to no index */
   if (common->index->len == 0) {
-    g_array_free (common->index, TRUE);
+    g_array_unref (common->index);
     common->index = NULL;
   }
 
@@ -3319,7 +3319,7 @@ gst_matroska_read_common_reset (GstElement * element,
 
   /* reset indexes */
   if (ctx->index) {
-    g_array_free (ctx->index, TRUE);
+    g_array_unref (ctx->index);
     ctx->index = NULL;
   }