[FFmpeg-cvslog] xxan: fix invalid memory access in xan_decode_frame_type0()
Anton Khirnov
git at videolan.org
Sun Apr 7 15:38:16 CEST 2013
ffmpeg | branch: release/0.10 | Anton Khirnov <anton at khirnov.net> | Wed Mar 6 09:06:16 2013 +0100| [e10af023b2579791d4de7a16d4958229dc62be03] | committer: Reinhard Tartler
xxan: fix invalid memory access in xan_decode_frame_type0()
The loop a few lines below the xan_unpack() call accesses up to
dec_size * 2 bytes into y_buffer, so dec_size must be limited to
buffer_size / 2.
CC:libav-stable at libav.org
(cherry picked from commit 8a49d2bcbe7573bb4b765728b2578fac0d19763f)
Signed-off-by: Reinhard Tartler <siretart at tauware.de>
(cherry picked from commit 62a657de168cf501acb23d48cc1aa00793dc83f3)
Signed-off-by: Reinhard Tartler <siretart at tauware.de>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=e10af023b2579791d4de7a16d4958229dc62be03
---
libavcodec/xxan.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/xxan.c b/libavcodec/xxan.c
index 0a37d48..59e1229 100644
--- a/libavcodec/xxan.c
+++ b/libavcodec/xxan.c
@@ -296,7 +296,7 @@ static int xan_decode_frame_type0(AVCodecContext *avctx)
if (chroma_off > corr_off)
corr_end = chroma_off;
bytestream2_seek(&s->gb, 8 + corr_off, SEEK_SET);
- dec_size = xan_unpack(s, s->scratch_buffer, s->buffer_size);
+ dec_size = xan_unpack(s, s->scratch_buffer, s->buffer_size / 2);
if (dec_size < 0)
dec_size = 0;
for (i = 0; i < dec_size; i++)
More information about the ffmpeg-cvslog
mailing list