summaryrefslogtreecommitdiff
path: root/libavformat
diff options
context:
space:
mode:
authorAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>2016-11-13 18:22:12 +0100
committerAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>2016-11-13 19:18:14 +0100
commit25012c56448a48487cdc9699465e640871dbcd60 (patch)
tree78d9e2bd291296def65dede120cde1712c7243ac /libavformat
parent03cef34aa66662e2ab3681d290e7c5a6634f4058 (diff)
downloadffmpeg-25012c56448a48487cdc9699465e640871dbcd60.tar.gz
filmstripdec: correctly check image dimensions
This prevents a division by zero in read_packet. Reviewed-by: Paul B Mahol <onemda@gmail.com> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Diffstat (limited to 'libavformat')
-rw-r--r--libavformat/filmstripdec.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/libavformat/filmstripdec.c b/libavformat/filmstripdec.c
index 414e276bfe..0aeb5947b0 100644
--- a/libavformat/filmstripdec.c
+++ b/libavformat/filmstripdec.c
@@ -25,6 +25,7 @@
*/
#include "libavutil/intreadwrite.h"
+#include "libavutil/imgutils.h"
#include "avformat.h"
#include "internal.h"
@@ -68,10 +69,8 @@ static int read_header(AVFormatContext *s)
st->codecpar->height = avio_rb16(pb);
film->leading = avio_rb16(pb);
- if (st->codecpar->width * 4LL * st->codecpar->height >= INT_MAX) {
- av_log(s, AV_LOG_ERROR, "dimensions too large\n");
- return AVERROR_PATCHWELCOME;
- }
+ if (av_image_check_size(st->codecpar->width, st->codecpar->height, 0, s) < 0)
+ return AVERROR_INVALIDDATA;
avpriv_set_pts_info(st, 64, 1, avio_rb16(pb));