[FFmpeg-cvslog] mpegvideo: Drop a faulty assert
Luca Barbato
git at videolan.org
Wed Oct 23 13:12:49 CEST 2013
ffmpeg | branch: master | Luca Barbato <lu_zero at gentoo.org> | Tue Oct 22 19:17:10 2013 +0200| [72072bf9de3241848ea86f68d2297b7a5d6ad49b] | committer: Luca Barbato
mpegvideo: Drop a faulty assert
That check is easily reachable by faulty input.
CC:libav-stable at libav.org
Reported-by: Torsten Sadowski <tsadowski at gmx.net>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=72072bf9de3241848ea86f68d2297b7a5d6ad49b
---
libavcodec/mpegvideo.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c
index 192e27f..718f946 100644
--- a/libavcodec/mpegvideo.c
+++ b/libavcodec/mpegvideo.c
@@ -1633,8 +1633,12 @@ int ff_MPV_frame_start(MpegEncContext *s, AVCodecContext *avctx)
return ret;
}
- assert(s->pict_type == AV_PICTURE_TYPE_I || (s->last_picture_ptr &&
- s->last_picture_ptr->f.data[0]));
+ if (s->pict_type != AV_PICTURE_TYPE_I &&
+ !(s->last_picture_ptr && s->last_picture_ptr->f.data[0])) {
+ av_log(s, AV_LOG_ERROR,
+ "Non-reference picture received and no reference available\n");
+ return AVERROR_INVALIDDATA;
+ }
if (s->picture_structure!= PICT_FRAME) {
int i;
More information about the ffmpeg-cvslog
mailing list