diff options
author | Vittorio Giovara <vittorio.giovara@gmail.com> | 2014-08-25 20:40:19 -0400 |
---|---|---|
committer | Vittorio Giovara <vittorio.giovara@gmail.com> | 2014-08-26 13:31:29 -0400 |
commit | e87f5e4e5f2e2e36b0b7826d708cda7049877af0 (patch) | |
tree | c9e579f4bffe807245980b307c86b55e7853ad4d /libavcodec | |
parent | 0a024268261d05ccdcf7e03c85fb78d22037a464 (diff) | |
download | ffmpeg-streaming-e87f5e4e5f2e2e36b0b7826d708cda7049877af0.zip ffmpeg-streaming-e87f5e4e5f2e2e36b0b7826d708cda7049877af0.tar.gz |
h264: fully check cropping amount from sps
Even if sps.crop is true, the cropping amount may be zero.
Fixes a sample with a valid but broken container cropping.
Diffstat (limited to 'libavcodec')
-rw-r--r-- | libavcodec/h264_slice.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libavcodec/h264_slice.c b/libavcodec/h264_slice.c index 83d3426..69aed1d 100644 --- a/libavcodec/h264_slice.c +++ b/libavcodec/h264_slice.c @@ -1028,9 +1028,11 @@ static int init_dimensions(H264Context *h) { int width = h->width - (h->sps.crop_right + h->sps.crop_left); int height = h->height - (h->sps.crop_top + h->sps.crop_bottom); + int crop_present = h->sps.crop_left || h->sps.crop_top || + h->sps.crop_right || h->sps.crop_bottom; /* handle container cropping */ - if (!h->sps.crop && + if (!crop_present && FFALIGN(h->avctx->width, 16) == h->width && FFALIGN(h->avctx->height, 16) == h->height) { width = h->avctx->width; |