[FFmpeg-cvslog] avcodec/tiff: move bpp check to after "end:"
Michael Niedermayer
git at videolan.org
Thu Mar 12 18:05:42 CET 2015
ffmpeg | branch: release/0.10 | Michael Niedermayer <michaelni at gmx.at> | Sun Mar 8 23:27:43 2015 +0100| [25e2ef355d8a5f2bb093811cee0aac19cc4889f8] | committer: Michael Niedermayer
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 at gmx.at>
(cherry picked from commit d5e9fc782150d4596c72440a0aa02b7f4f1254b1)
Conflicts:
libavcodec/tiff.c
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=25e2ef355d8a5f2bb093811cee0aac19cc4889f8
---
libavcodec/tiff.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/libavcodec/tiff.c b/libavcodec/tiff.c
index 930da9f..d6e2295 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;
}
More information about the ffmpeg-cvslog
mailing list