aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKostya Shishkov <kostya.shishkov@gmail.com>2017-06-08 18:05:26 +0200
committerKostya Shishkov <kostya.shishkov@gmail.com>2017-06-08 18:05:26 +0200
commitf149a5b7c236d8ab88daca8417a0cc95dda4edfc (patch)
treefbb81e6d8724d0cf7dc331a97a166bb59d6d28fc
parent836bf1506835bd14b3f5c52bd05df8257df7d96c (diff)
downloadnihav-f149a5b7c236d8ab88daca8417a0cc95dda4edfc.tar.gz
avi: skip list chunks correctly
-rw-r--r--src/demuxers/avi.rs10
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));
}