50 lines
1.8 KiB
Diff
50 lines
1.8 KiB
Diff
|
From d08f9f1de1782d30cb902df7fd3aab50134d1792 Mon Sep 17 00:00:00 2001
|
||
|
|
From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= <sebastian@centricular.com>
|
||
|
|
Date: Wed, 2 Oct 2024 14:44:21 +0300
|
||
|
|
Subject: [PATCH 03/28] gdkpixbufdec: Check if initializing the video info
|
||
|
|
actually succeeded
|
||
|
|
|
||
|
|
Otherwise a 0-byte buffer would be allocated, which gives NULL memory when
|
||
|
|
mapped.
|
||
|
|
|
||
|
|
Thanks to Antonio Morales for finding and reporting the issue.
|
||
|
|
|
||
|
|
Fixes GHSL-2024-118
|
||
|
|
Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/3876
|
||
|
|
|
||
|
|
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/8103>
|
||
|
|
---
|
||
|
|
.../gst-plugins-good/ext/gdk_pixbuf/gstgdkpixbufdec.c | 9 ++++++++-
|
||
|
|
1 file changed, 8 insertions(+), 1 deletion(-)
|
||
|
|
|
||
|
|
diff --git a/subprojects/gst-plugins-good/ext/gdk_pixbuf/gstgdkpixbufdec.c b/subprojects/gst-plugins-good/ext/gdk_pixbuf/gstgdkpixbufdec.c
|
||
|
|
index 5482998c0d..de5f054964 100644
|
||
|
|
--- a/subprojects/gst-plugins-good/ext/gdk_pixbuf/gstgdkpixbufdec.c
|
||
|
|
+++ b/subprojects/gst-plugins-good/ext/gdk_pixbuf/gstgdkpixbufdec.c
|
||
|
|
@@ -322,7 +322,8 @@ gst_gdk_pixbuf_dec_flush (GstGdkPixbufDec * filter)
|
||
|
|
|
||
|
|
|
||
|
|
gst_video_info_init (&info);
|
||
|
|
- gst_video_info_set_format (&info, fmt, width, height);
|
||
|
|
+ if (!gst_video_info_set_format (&info, fmt, width, height))
|
||
|
|
+ goto format_not_supported;
|
||
|
|
info.fps_n = filter->in_fps_n;
|
||
|
|
info.fps_d = filter->in_fps_d;
|
||
|
|
caps = gst_video_info_to_caps (&info);
|
||
|
|
@@ -384,6 +385,12 @@ channels_not_supported:
|
||
|
|
("%d channels not supported", n_channels));
|
||
|
|
return GST_FLOW_ERROR;
|
||
|
|
}
|
||
|
|
+format_not_supported:
|
||
|
|
+ {
|
||
|
|
+ GST_ELEMENT_ERROR (filter, STREAM, DECODE, (NULL),
|
||
|
|
+ ("%d channels with %dx%d not supported", n_channels, width, height));
|
||
|
|
+ return GST_FLOW_ERROR;
|
||
|
|
+ }
|
||
|
|
no_buffer:
|
||
|
|
{
|
||
|
|
GST_DEBUG ("Failed to create outbuffer - %s", gst_flow_get_name (ret));
|
||
|
|
--
|
||
|
|
2.47.0
|
||
|
|
|