diff options
author | Matt Wolenetz <wolenetz@chromium.org> | 2018-03-02 15:12:41 -0800 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2018-03-05 01:16:34 +0100 |
commit | 133ddd38750acc01d0a9599d5b31375d33798d67 (patch) | |
tree | 9defcb16b64d554ed1edbc25f8cdcdf75f654968 /libavdevice/timefilter.h | |
parent | 0ec7eb930520b21b9cda34809378b0a8ca3a46e2 (diff) | |
download | ffmpeg-133ddd38750acc01d0a9599d5b31375d33798d67.tar.gz |
avformat/mov: Initialize a potential gap in ctts_data in mov_build_index
mov_read_ctts ignores ctts entries having count <= 0. Generally, the
aggregate of all ctts entries' count fields resulting from mov_read_ctts
can be less than the corresponding sample_count.
mov_build_index attempts to normalize any existing ctts_data counts to
be 1, to make a 1-1 mapping of a ctts_data entry to a sample.
That 1-1 mapping left a tail of uninitialized ctts_data entries when the
aggregate, normalized ctts_count < sample_count.
Even more generally, later usage of ctts_data may depend on the entire
ctts_allocated_size having been initialized.
This change memsets the entire allocation of the normalized ctts_data in
mov_build_index, to prevent use of uninitialized data later.
BUG=816787
Change-Id: I7fd7db255e3aeed076ee32c90cb2df211741c052
Reviewed-on: https://chromium-review.googlesource.com/947110
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavdevice/timefilter.h')
0 files changed, 0 insertions, 0 deletions