diff options
author | Kostya Shishkov <kostya.shishkov@gmail.com> | 2017-06-08 18:05:26 +0200 |
---|---|---|
committer | Kostya Shishkov <kostya.shishkov@gmail.com> | 2017-06-08 18:05:26 +0200 |
commit | f149a5b7c236d8ab88daca8417a0cc95dda4edfc (patch) | |
tree | fbb81e6d8724d0cf7dc331a97a166bb59d6d28fc /src | |
parent | 836bf1506835bd14b3f5c52bd05df8257df7d96c (diff) | |
download | nihav-f149a5b7c236d8ab88daca8417a0cc95dda4edfc.tar.gz |
avi: skip list chunks correctly
Diffstat (limited to 'src')
-rw-r--r-- | src/demuxers/avi.rs | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/demuxers/avi.rs b/src/demuxers/avi.rs index 2a3f8f0..2d2a261 100644 --- a/src/demuxers/avi.rs +++ b/src/demuxers/avi.rs @@ -139,7 +139,8 @@ impl<'a> AVIDemuxer<'a> { if RIFFTag::Chunk(tag) == end_tag { return Ok((size, true)); } - let ltag = if is_list_tag(tag) { self.src.read_u32be()? } else { 0 }; + let is_list = is_list_tag(tag); + let ltag = if is_list { self.src.read_u32be()? } else { 0 }; if RIFFTag::List(tag, ltag) == end_tag { return Ok((size, true)); } @@ -170,7 +171,12 @@ impl<'a> AVIDemuxer<'a> { return Ok((size + 8, false)); } } - self.src.read_skip(size)?; + if !is_list { + self.src.read_skip(size)?; + } else { + if size < 4 { return Err(InvalidData); } + self.src.read_skip(size - 4)?; + } if (size & 1) == 1 { self.src.read_skip(1)?; } return Ok((size + 8, false)); } |