# HG changeset patch # User michael # Date 1260790486 0 # Node ID e5fac8ac71f63f446aa6b97ff1677a6da5876378 # Parent aa8bebc96764cfb7094428c8482ca6b6ca9bff00 Check sample_aspect_ratio validity for mpeg4/h263/h263p encoding. Merged from ffmbc0.3 diff -r aa8bebc96764 -r e5fac8ac71f6 mpegvideo_enc.c --- a/mpegvideo_enc.c Mon Dec 14 11:18:28 2009 +0000 +++ b/mpegvideo_enc.c Mon Dec 14 11:34:46 2009 +0000 @@ -395,6 +395,14 @@ return -1; } + if ((s->codec_id == CODEC_ID_MPEG4 || s->codec_id == CODEC_ID_H263 || + s->codec_id == CODEC_ID_H263P) && + (avctx->sample_aspect_ratio.num > 255 || avctx->sample_aspect_ratio.den > 255)) { + av_log(avctx, AV_LOG_ERROR, "Invalid pixel aspect ratio %i/%i, limit is 255/255\n", + avctx->sample_aspect_ratio.num, avctx->sample_aspect_ratio.den); + return -1; + } + if((s->flags & (CODEC_FLAG_INTERLACED_DCT|CODEC_FLAG_INTERLACED_ME|CODEC_FLAG_ALT_SCAN)) && s->codec_id != CODEC_ID_MPEG4 && s->codec_id != CODEC_ID_MPEG2VIDEO){ av_log(avctx, AV_LOG_ERROR, "interlacing not supported by codec\n");