There should be no case that needs this and its a potential for
creating corner cases

Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
---
 libavutil/frame.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavutil/frame.c b/libavutil/frame.c
index 8811dcdcfe..69ee240758 100644
--- a/libavutil/frame.c
+++ b/libavutil/frame.c
@@ -668,6 +668,8 @@ AVFrameSideData *av_frame_new_side_data(AVFrame *frame,
                                         enum AVFrameSideDataType type,
                                         int size)
 {
+    if (size <= 0)
+        return NULL;
 
     return frame_new_side_data(frame, type, av_buffer_alloc(size));
 }
-- 
2.11.0

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to