aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michaelni@gmx.at>2013-01-07 15:47:21 +0100
committerMichael Niedermayer <michaelni@gmx.at>2013-01-20 02:45:35 +0100
commit95f8701c32d8737afe5185e6b2f0fc54d976719f (patch)
tree2f2c6274067a9fa577355aaa332b906ae5a758b7
parent5af41713046efed635d202a2d9b641c626561d71 (diff)
downloadffmpeg-95f8701c32d8737afe5185e6b2f0fc54d976719f.tar.gz
mpeg12enc: check dimension validity
Signed-off-by: Michael Niedermayer <michaelni@gmx.at> (cherry picked from commit 7fb87bc5f24b1be13269109506c05e4c54695b5e) Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
-rw-r--r--libavcodec/mpeg12enc.c13
1 files changed, 13 insertions, 0 deletions
diff --git a/libavcodec/mpeg12enc.c b/libavcodec/mpeg12enc.c
index 9726a30ae5..9e31543995 100644
--- a/libavcodec/mpeg12enc.c
+++ b/libavcodec/mpeg12enc.c
@@ -167,6 +167,19 @@ static av_cold int encode_init(AVCodecContext *avctx)
}
}
+ if ((avctx->width & 0xFFF) == 0 && (avctx->height & 0xFFF) == 1) {
+ av_log(avctx, AV_LOG_ERROR, "Width / Height is invalid for MPEG2\n");
+ return AVERROR(EINVAL);
+ }
+
+ if (s->strict_std_compliance > FF_COMPLIANCE_UNOFFICIAL) {
+ if ((avctx->width & 0xFFF) == 0 || (avctx->height & 0xFFF) == 0) {
+ av_log(avctx, AV_LOG_ERROR, "Width or Height are not allowed to be multiplies of 4096\n"
+ "add '-strict %d' if you want to use them anyway.\n", FF_COMPLIANCE_UNOFFICIAL);
+ return AVERROR(EINVAL);
+ }
+ }
+
s->drop_frame_timecode = s->drop_frame_timecode || !!(avctx->flags2 & CODEC_FLAG2_DROP_FRAME_TIMECODE);
if (s->drop_frame_timecode)
s->tc.flags |= AV_TIMECODE_FLAG_DROPFRAME;