diff options
| author | Xiaohan Wang <xhwang@chromium.org> | 2014-11-06 12:59:54 -0800 |
|---|---|---|
| committer | Vittorio Giovara <vittorio.giovara@gmail.com> | 2015-01-27 14:32:56 +0000 |
| commit | 532c96a2158c04f265d750d54f2f103b8d9fe0ef (patch) | |
| tree | 8381f7d44073c8983ea8cc3d1f4ed38c290ad81b | |
| parent | 891de4b27a07b808839b9e873b6a886248c8fd6b (diff) | |
| download | ffmpeg-532c96a2158c04f265d750d54f2f103b8d9fe0ef.tar.gz | |
matroskadec: Fix read-after-free in matroska_read_seek()
In matroska_read_seek(), |tracks| is assigned at the begining of the
function. However, functions like matroska_parse_cues() could reallocate
the tracks and invalidate |tracks|.
This assigns |tracks| only before using it, so that it will not get
invalidated elsewhere.
Bug-Id: chromium/427266
| -rw-r--r-- | libavformat/matroskadec.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c index 59fc34b142..f3844582e2 100644 --- a/libavformat/matroskadec.c +++ b/libavformat/matroskadec.c @@ -2532,7 +2532,7 @@ static int matroska_read_seek(AVFormatContext *s, int stream_index, int64_t timestamp, int flags) { MatroskaDemuxContext *matroska = s->priv_data; - MatroskaTrack *tracks = matroska->tracks.elem; + MatroskaTrack *tracks = NULL; AVStream *st = s->streams[stream_index]; int i, index, index_sub, index_min; @@ -2562,6 +2562,7 @@ static int matroska_read_seek(AVFormatContext *s, int stream_index, return 0; index_min = index; + tracks = matroska->tracks.elem; for (i = 0; i < matroska->tracks.nb_elem; i++) { tracks[i].audio.pkt_cnt = 0; tracks[i].audio.sub_packet_cnt = 0; |