[FFmpeg-cvslog] avutil/frame: Disallow zero sized frame side data

Michael Niedermayer git at videolan.org
Sat Feb 25 23:17:39 EET 2017


ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Thu Feb 23 15:19:32 2017 +0100| [7e4f32f4e4b93c95dcc872cb844c5548e69f352e] | committer: Michael Niedermayer

avutil/frame: Disallow zero sized frame side data

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

Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=7e4f32f4e4b93c95dcc872cb844c5548e69f352e
---

 libavutil/frame.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavutil/frame.c b/libavutil/frame.c
index 8811dcd..69ee240 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));
 }



More information about the ffmpeg-cvslog mailing list