[FFmpeg-devel] [PATCH] exr: fix header parsing

Jean First jeanfirst at gmail.com
Tue May 15 16:34:58 CEST 2012


the header in the sample provided for ticket #1306 is not parsed correctly and thus
ffmpeg tries to decode the sample instead of abording the decoding.

I tested it with two other exr samples I have - one float, one half float - and
they still decode correctly. Maybe someone else with more sample can give it a
spin.

Signed-off-by: Jean First <jeanfirst at gmail.com>
---
 libavcodec/exr.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/exr.c b/libavcodec/exr.c
index 3a93f22..0ef8db5 100644
--- a/libavcodec/exr.c
+++ b/libavcodec/exr.c
@@ -198,7 +198,7 @@ static int decode_frame(AVCodecContext *avctx,
             if (!variable_buffer_data_size)
                 return -1;
 
-            channel_list_end = buf + variable_buffer_data_size + 4;
+            channel_list_end = buf + variable_buffer_data_size;
             while (channel_list_end - buf >= 19) {
                 int current_bits_per_color_id = -1;
                 int channel_index = -1;
-- 
1.7.9.4



More information about the ffmpeg-devel mailing list