mxfdemux: Don't leak index table segments on failures

The segment was freed ... but not the contents :)

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

diff --git a/subprojects/gst-plugins-bad/gst/mxf/mxfdemux.c b/subprojects/gst-plugins-bad/gst/mxf/mxfdemux.c
index f9d9a69..d9eb9a5 100644 (file)
--- a/subprojects/gst-plugins-bad/gst/mxf/mxfdemux.c
+++ b/subprojects/gst-plugins-bad/gst/mxf/mxfdemux.c
@@ -3372,6 +3372,7 @@ gst_mxf_demux_handle_index_table_segment (GstMXFDemux * demux, GstMXFKLV * klv)
   if (g_list_find_custom (demux->pending_index_table_segments, segment,
           (GCompareFunc) compare_index_table_segment)) {
     GST_DEBUG_OBJECT (demux, "Already in pending list");
+    mxf_index_table_segment_reset (segment);
     g_free (segment);
     return GST_FLOW_OK;
   }
@@ -3380,6 +3381,7 @@ gst_mxf_demux_handle_index_table_segment (GstMXFDemux * demux, GstMXFKLV * klv)
     if (g_array_binary_search (table->segments, segment,
             (GCompareFunc) compare_index_table_segment, NULL)) {
       GST_DEBUG_OBJECT (demux, "Already handled");
+      mxf_index_table_segment_reset (segment);
       g_free (segment);
       return GST_FLOW_OK;
     }