[FFmpeg-cvslog] xxan: don't read before start of buffer in av_memcpy_backptr().

Ronald S. Bultje git at videolan.org
Mon Apr 2 01:45:29 CEST 2012


ffmpeg | branch: release/0.8 | Ronald S. Bultje <rsbultje at gmail.com> | Thu Mar  8 16:32:46 2012 -0800| [5ae49ddaa447bb4fba287f92ca508caba399ffbd] | committer: Reinhard Tartler

xxan: don't read before start of buffer in av_memcpy_backptr().

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable at libav.org
(cherry picked from commit f1279e286b00e99f343adb51e251f036a3df6f32)

Signed-off-by: Reinhard Tartler <siretart at tauware.de>

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

 libavcodec/xxan.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/libavcodec/xxan.c b/libavcodec/xxan.c
index 28c868d..daaba63 100644
--- a/libavcodec/xxan.c
+++ b/libavcodec/xxan.c
@@ -129,7 +129,8 @@ static int xan_unpack(uint8_t *dest, const int dest_len,
                 if (size + size2 > dest_end - dest)
                     break;
             }
-            if (src + size > src_end || dest + size + size2 > dest_end)
+            if (src + size > src_end || dest + size + size2 > dest_end ||
+                dest - orig_dest + size < back)
                 return -1;
             bytestream_get_buffer(&src, dest, size);
             dest += size;



More information about the ffmpeg-cvslog mailing list