diff options
author | Michael Niedermayer <michaelni@gmx.at> | 2015-03-08 23:27:43 +0100 |
---|---|---|
committer | Michael Niedermayer <michaelni@gmx.at> | 2015-03-12 18:03:50 +0100 |
commit | 25e2ef355d8a5f2bb093811cee0aac19cc4889f8 (patch) | |
tree | f4e3c4dd3086e1702ff4aa1a104b95e139882371 | |
parent | 7f99fae1ec2903366ecc529d84d2542daa5f3c57 (diff) | |
download | ffmpeg-25e2ef355d8a5f2bb093811cee0aac19cc4889f8.tar.gz |
avcodec/tiff: move bpp check to after "end:"
This ensures that all current and future code-pathes get bpp checked
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
(cherry picked from commit d5e9fc782150d4596c72440a0aa02b7f4f1254b1)
Conflicts:
libavcodec/tiff.c
-rw-r--r-- | libavcodec/tiff.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c index 930da9ff3e..d6e2295564 100644 --- a/libavcodec/tiff.c +++ b/libavcodec/tiff.c @@ -407,13 +407,6 @@ static int tiff_decode_tag(TiffContext *s) s->bpp = -1; } } - if (s->bpp > 64U) { - av_log(s->avctx, AV_LOG_ERROR, - "This format is not supported (bpp=%d, %d components)\n", - s->bpp, count); - s->bpp = 0; - return AVERROR_INVALIDDATA; - } break; case TIFF_SAMPLES_PER_PIXEL: if (count != 1) { @@ -556,6 +549,13 @@ static int tiff_decode_tag(TiffContext *s) default: av_log(s->avctx, AV_LOG_DEBUG, "Unknown or unsupported tag %d/0X%0X\n", tag, tag); } + if (s->bpp > 64U) { + av_log(s->avctx, AV_LOG_ERROR, + "This format is not supported (bpp=%d, %d components)\n", + s->bpp, count); + s->bpp = 0; + return AVERROR_INVALIDDATA; + } bytestream2_seek(&s->gb, start, SEEK_SET); return 0; } |