diff options
author | Chris Evans <cevans@chromium.org> | 2012-01-05 21:19:30 +0100 |
---|---|---|
committer | Reinhard Tartler <siretart@tauware.de> | 2012-01-07 22:03:48 +0100 |
commit | 90a4a467477be8c292daa08a9516ee78ca0d517b (patch) | |
tree | 3e42b583a348ee26ce6a3adb672d106bcdf9bfb9 | |
parent | 6d6254ba9fbb22260939c06db1faed5bbd295ad4 (diff) | |
download | ffmpeg-90a4a467477be8c292daa08a9516ee78ca0d517b.tar.gz |
matroskadec: Fix a bug where a pointer was cached to an array that might later move due to a realloc()
Fixes bug #190
Chromium bug #100492
related to CVE-2011-3893
Signed-off-by: Reinhard Tartler <siretart@tauware.de>
(cherry-picked from commit faaec4676cb4c7a2303d50df66c6290bc96a7657)
Signed-off-by: Reinhard Tartler <siretart@tauware.de>
(cherry picked from commit 1f625431e2bb9564760fba3ab8077ae07ce7c7a1)
Signed-off-by: Reinhard Tartler <siretart@tauware.de>
-rw-r--r-- | libavformat/matroskadec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 9745585ca5..ab42e0a70e 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matroskadec.c @@ -1074,13 +1074,13 @@ static void matroska_convert_tags(AVFormatContext *s) static void matroska_execute_seekhead(MatroskaDemuxContext *matroska) { EbmlList *seekhead_list = &matroska->seekhead; - MatroskaSeekhead *seekhead = seekhead_list->elem; uint32_t level_up = matroska->level_up; int64_t before_pos = url_ftell(matroska->ctx->pb); MatroskaLevel level; int i; for (i=0; i<seekhead_list->nb_elem; i++) { + MatroskaSeekhead *seekhead = seekhead_list->elem; int64_t offset = seekhead[i].pos + matroska->segment_start; if (seekhead[i].pos <= before_pos |