[PATCH] winegstreamer: Check the format GUID, size, and pointer in amt_to_gst_caps().

Zebediah Figura z.figura12 at gmail.com
Sun Apr 5 19:43:35 CDT 2020


Some application filters enumerate types without a format block.

Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
---
 dlls/winegstreamer/gstdemux.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/dlls/winegstreamer/gstdemux.c b/dlls/winegstreamer/gstdemux.c
index 2d9efe8ead..2186192b59 100644
--- a/dlls/winegstreamer/gstdemux.c
+++ b/dlls/winegstreamer/gstdemux.c
@@ -387,6 +387,10 @@ static GstCaps *amt_to_gst_caps_video(const AM_MEDIA_TYPE *mt)
     unsigned int i;
     GstCaps *caps;
 
+    if (!IsEqualGUID(&mt->formattype, &FORMAT_VideoInfo)
+            || mt->cbFormat < sizeof(VIDEOINFOHEADER) || !mt->pbFormat)
+        return NULL;
+
     for (i = 0; i < ARRAY_SIZE(format_map); ++i)
     {
         if (IsEqualGUID(&mt->subtype, format_map[i].subtype))
@@ -425,6 +429,10 @@ static GstCaps *amt_to_gst_caps_audio(const AM_MEDIA_TYPE *mt)
     GstAudioFormat format = GST_AUDIO_FORMAT_UNKNOWN;
     GstAudioInfo info;
 
+    if (!IsEqualGUID(&mt->formattype, &FORMAT_WaveFormatEx)
+            || mt->cbFormat < sizeof(WAVEFORMATEX) || !mt->pbFormat)
+        return NULL;
+
     if (IsEqualGUID(&mt->subtype, &MEDIASUBTYPE_PCM))
         format = gst_audio_format_build_integer(wfx->wBitsPerSample != 8,
                 G_LITTLE_ENDIAN, wfx->wBitsPerSample, wfx->wBitsPerSample);
-- 
2.26.0




More information about the wine-devel mailing list