[FFmpeg-cvslog] exr: check version too

Paul B Mahol git at videolan.org
Fri Feb 22 20:21:17 CET 2013


ffmpeg | branch: master | Paul B Mahol <onemda at gmail.com> | Wed Feb 20 10:32:22 2013 +0000| [3456891291024d97f167a5eea82b39e53a008843] | committer: Paul B Mahol

exr: check version too

Separate version from flags.

Signed-off-by: Paul B Mahol <onemda at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=3456891291024d97f167a5eea82b39e53a008843
---

 libavcodec/exr.c |   12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/libavcodec/exr.c b/libavcodec/exr.c
index 46f816a..53b8e5c 100644
--- a/libavcodec/exr.c
+++ b/libavcodec/exr.c
@@ -236,7 +236,7 @@ static int decode_frame(AVCodecContext *avctx,
     AVFrame *const p = &s->picture;
     uint8_t *ptr;
 
-    int i, x, y, stride, magic_number, version_flag, ret;
+    int i, x, y, stride, magic_number, version, flags, ret;
     int w = 0;
     int h = 0;
     unsigned int xmin   = ~0;
@@ -271,8 +271,14 @@ static int decode_frame(AVCodecContext *avctx,
         return AVERROR_INVALIDDATA;
     }
 
-    version_flag = bytestream_get_le32(&buf);
-    if ((version_flag & 0x200) == 0x200) {
+    version = bytestream_get_byte(&buf);
+    if (version != 2) {
+        av_log(avctx, AV_LOG_ERROR, "Unsupported version %d\n", version);
+        return AVERROR_PATCHWELCOME;
+    }
+
+    flags = bytestream_get_le24(&buf);
+    if (flags & 0x2) {
         av_log(avctx, AV_LOG_ERROR, "Tile based images are not supported\n");
         return AVERROR_PATCHWELCOME;
     }



More information about the ffmpeg-cvslog mailing list