diff options
| author | acolwell@chromium.org <acolwell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-20 08:40:30 +0000 |
|---|---|---|
| committer | acolwell@chromium.org <acolwell@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-20 08:40:30 +0000 |
| commit | d749be45057dabc88a2346698794669472fcf2a8 (patch) | |
| tree | 51b21c2c06f49f606d7202a71a8ca8599be83c0d /ipc/ipc_channel_unittest.cc | |
| parent | fc43f5a21a96d4cfd4ccd45877538e768373d6b2 (diff) | |
| download | chromium_src-d749be45057dabc88a2346698794669472fcf2a8.zip chromium_src-d749be45057dabc88a2346698794669472fcf2a8.tar.gz chromium_src-d749be45057dabc88a2346698794669472fcf2a8.tar.bz2 | |
Fix muxed MP4 parsing so it won't crash on partial media segment appends.
Partial appends could cause the MP4StreamParser to emit buffers in
non-monotonically increasing timestamp order because of how we process
trun boxes. This patch makes sure that processing of trun boxes is
deferred until we have all the sample & aux_info data before emitting
samples. This prevents the parser from emitting samples in such a way
that will break downstream code. This is a minimal impact fix to
avoid bad behavior at the expense of buffering more data.
BUG=None
TEST=All MP4StreamParserTests that do partial appends now verify the fix.
Review URL: https://codereview.chromium.org/348623003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278650 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ipc/ipc_channel_unittest.cc')
0 files changed, 0 insertions, 0 deletions