ffmpeg | branch: master | Michael Niedermayer <[email protected]> | Fri 
Feb 26 12:59:08 2016 +0100| [d39b770aa27687da3b799252e614caa8b8686c13] | 
committer: Michael Niedermayer

avcodec/utils: Check that the video data[] arrays are NULL on the input to 
get_buffer_internal()

This should return an error to the decoder if the struct it tried to getbuffer 
is dirty

Reviewed-by: Reimar Döffinger <[email protected]>
Signed-off-by: Michael Niedermayer <[email protected]>

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

 libavcodec/utils.c |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index b5de8a1..5791ffa 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -870,6 +870,11 @@ static int get_buffer_internal(AVCodecContext *avctx, 
AVFrame *frame, int flags)
             frame->height = FFMAX(avctx->height, 
AV_CEIL_RSHIFT(avctx->coded_height, avctx->lowres));
             override_dimensions = 0;
         }
+
+        if (frame->data[0] || frame->data[1] || frame->data[2] || 
frame->data[3]) {
+            av_log(avctx, AV_LOG_ERROR, "pic->data[*]!=NULL in 
get_buffer_internal\n");
+            return AVERROR(EINVAL);
+        }
     }
     ret = ff_decode_frame_props(avctx, frame);
     if (ret < 0)

_______________________________________________
ffmpeg-cvslog mailing list
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-cvslog

Reply via email to