From: Emil Velikov <[email protected]>

Assert in the very unlikely case when GST_VIDEO_INFO_N_PLANES returns
more than MAX_NUM_PLANES. Otherwise we'll get random memory corruption.

Signed-off-by: Emil Velikov <[email protected]>
---
 gst-decoder.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/gst-decoder.c b/gst-decoder.c
index dd2a6fa..d3cdd9f 100644
--- a/gst-decoder.c
+++ b/gst-decoder.c
@@ -368,6 +368,8 @@ buffer_to_image(struct decoder *dec, GstBuffer *buf)
        GstMemory *mem;
        int dmabuf_fd = -1;
 
+       assert(nplanes != MAX_NUM_PLANES);
+
        /* Query gst_is_dmabuf_memory() here, since the gstmemory
         * block might get merged below by gst_buffer_map(), meaning
         * that the mem pointer would become invalid */
-- 
2.12.2

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to