From c0dceda8e969f74f2326539c1f0368c2fd7afcd7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= Date: Mon, 30 Sep 2024 16:32:48 +0300 Subject: [PATCH] matroskademux: Only unmap GstMapInfo in WavPack header extraction error paths if previously mapped Thanks to Antonio Morales for finding and reporting the issue. Fixes GHSL-2024-197 Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/3863 Part-of: --- subprojects/gst-plugins-good/gst/matroska/matroska-demux.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/subprojects/gst-plugins-good/gst/matroska/matroska-demux.c b/subprojects/gst-plugins-good/gst/matroska/matroska-demux.c index 41725e83a6..9e0de058e6 100644 --- a/subprojects/gst-plugins-good/gst/matroska/matroska-demux.c +++ b/subprojects/gst-plugins-good/gst/matroska/matroska-demux.c @@ -3891,7 +3891,6 @@ gst_matroska_demux_add_wvpk_header (GstElement * element, GstMatroskaTrackAudioContext *audiocontext = (GstMatroskaTrackAudioContext *) stream; GstBuffer *newbuf = NULL; - GstMapInfo map, outmap; guint8 *buf_data, *data; Wavpack4Header wvh; @@ -3908,11 +3907,11 @@ gst_matroska_demux_add_wvpk_header (GstElement * element, if (audiocontext->channels <= 2) { guint32 block_samples, tmp; + GstMapInfo outmap; gsize size = gst_buffer_get_size (*buf); if (size < 4) { GST_ERROR_OBJECT (element, "Too small wavpack buffer"); - gst_buffer_unmap (*buf, &map); return GST_FLOW_ERROR; } @@ -3950,6 +3949,7 @@ gst_matroska_demux_add_wvpk_header (GstElement * element, *buf = newbuf; audiocontext->wvpk_block_index += block_samples; } else { + GstMapInfo map, outmap; guint8 *outdata = NULL; gsize buf_size, size; guint32 block_samples, flags, crc;