1
0
Fork 0
mirror of https://github.com/alfg/mp4-rust.git synced 2024-05-20 01:08:06 +00:00
mp4-rust/src/mp4box/trun.rs
oftheforest 7cfdffbd71
Fix several overflows in box and track processing (#94)
* Fix several overflows in box and track processing

* Use size_of::<Type>() instead of magic numbers

* Fix a panic in Mp4Track::read_sample() for one-past-the-end

This appears to be a bug unmasked by other changes. read_sample() calls
sample_offset() then sample_size(), and assumes that if the former returns Ok
then the latter does as well. However, if the sample_id is one past the end,
sample_offset() might succeed (it only checks samples _up to_ the given
sample_id but not _including_ it) while sample_size() fails (because the sample
doesn't exist). read_sample() will then panic.

Fix this by duplicating the error propagation (that is currently done for
sample_offset) for sample_size, instead of unwrapping. This is a cautious
change that fixes the bug; alternatively, having sample_offset() call
sample_size() on the given sample_id and propagate any error might also work.

* Account for header size in box processing overflow fixes

* Ensure that boxes aren't bigger than their containers

Together with the entry_count checks, this eliminates several OOMs when reading
incorrect mp4 files.

* Fix order of arithmetic operations

This was due to an incorrect transcription when switching to checked
arithmetic, and fixes a bug that could cause attempted lookups of the wrong
chunk_id.
2023-02-18 11:46:51 -08:00

271 lines
8.9 KiB
Rust

use byteorder::{BigEndian, ReadBytesExt, WriteBytesExt};
use serde::Serialize;
use std::io::{Read, Seek, Write};
use std::mem::size_of;
use crate::mp4box::*;
#[derive(Debug, Clone, PartialEq, Eq, Default, Serialize)]
pub struct TrunBox {
pub version: u8,
pub flags: u32,
pub sample_count: u32,
pub data_offset: Option<i32>,
pub first_sample_flags: Option<u32>,
#[serde(skip_serializing)]
pub sample_durations: Vec<u32>,
#[serde(skip_serializing)]
pub sample_sizes: Vec<u32>,
#[serde(skip_serializing)]
pub sample_flags: Vec<u32>,
#[serde(skip_serializing)]
pub sample_cts: Vec<u32>,
}
impl TrunBox {
pub const FLAG_DATA_OFFSET: u32 = 0x01;
pub const FLAG_FIRST_SAMPLE_FLAGS: u32 = 0x04;
pub const FLAG_SAMPLE_DURATION: u32 = 0x100;
pub const FLAG_SAMPLE_SIZE: u32 = 0x200;
pub const FLAG_SAMPLE_FLAGS: u32 = 0x400;
pub const FLAG_SAMPLE_CTS: u32 = 0x800;
pub fn get_type(&self) -> BoxType {
BoxType::TrunBox
}
pub fn get_size(&self) -> u64 {
let mut sum = HEADER_SIZE + HEADER_EXT_SIZE + 4;
if TrunBox::FLAG_DATA_OFFSET & self.flags > 0 {
sum += 4;
}
if TrunBox::FLAG_FIRST_SAMPLE_FLAGS & self.flags > 0 {
sum += 4;
}
if TrunBox::FLAG_SAMPLE_DURATION & self.flags > 0 {
sum += 4 * self.sample_count as u64;
}
if TrunBox::FLAG_SAMPLE_SIZE & self.flags > 0 {
sum += 4 * self.sample_count as u64;
}
if TrunBox::FLAG_SAMPLE_FLAGS & self.flags > 0 {
sum += 4 * self.sample_count as u64;
}
if TrunBox::FLAG_SAMPLE_CTS & self.flags > 0 {
sum += 4 * self.sample_count as u64;
}
sum
}
}
impl Mp4Box for TrunBox {
fn box_type(&self) -> BoxType {
self.get_type()
}
fn box_size(&self) -> u64 {
self.get_size()
}
fn to_json(&self) -> Result<String> {
Ok(serde_json::to_string(&self).unwrap())
}
fn summary(&self) -> Result<String> {
let s = format!("sample_size={}", self.sample_count);
Ok(s)
}
}
impl<R: Read + Seek> ReadBox<&mut R> for TrunBox {
fn read_box(reader: &mut R, size: u64) -> Result<Self> {
let start = box_start(reader)?;
let (version, flags) = read_box_header_ext(reader)?;
let header_size = HEADER_SIZE + HEADER_EXT_SIZE;
let other_size = size_of::<u32>() // sample_count
+ if TrunBox::FLAG_DATA_OFFSET & flags > 0 { size_of::<i32>() } else { 0 } // data_offset
+ if TrunBox::FLAG_FIRST_SAMPLE_FLAGS & flags > 0 { size_of::<u32>() } else { 0 }; // first_sample_flags
let sample_size = if TrunBox::FLAG_SAMPLE_DURATION & flags > 0 { size_of::<u32>() } else { 0 } // sample_duration
+ if TrunBox::FLAG_SAMPLE_SIZE & flags > 0 { size_of::<u32>() } else { 0 } // sample_size
+ if TrunBox::FLAG_SAMPLE_FLAGS & flags > 0 { size_of::<u32>() } else { 0 } // sample_flags
+ if TrunBox::FLAG_SAMPLE_CTS & flags > 0 { size_of::<u32>() } else { 0 }; // sample_composition_time_offset
let sample_count = reader.read_u32::<BigEndian>()?;
let data_offset = if TrunBox::FLAG_DATA_OFFSET & flags > 0 {
Some(reader.read_i32::<BigEndian>()?)
} else {
None
};
let first_sample_flags = if TrunBox::FLAG_FIRST_SAMPLE_FLAGS & flags > 0 {
Some(reader.read_u32::<BigEndian>()?)
} else {
None
};
let mut sample_durations = Vec::new();
let mut sample_sizes = Vec::new();
let mut sample_flags = Vec::new();
let mut sample_cts = Vec::new();
if u64::from(sample_count) * sample_size as u64
> size
.saturating_sub(header_size)
.saturating_sub(other_size as u64)
{
return Err(Error::InvalidData(
"trun sample_count indicates more values than could fit in the box",
));
}
if TrunBox::FLAG_SAMPLE_DURATION & flags > 0 {
sample_durations.reserve(sample_count as usize);
}
if TrunBox::FLAG_SAMPLE_SIZE & flags > 0 {
sample_sizes.reserve(sample_count as usize);
}
if TrunBox::FLAG_SAMPLE_FLAGS & flags > 0 {
sample_flags.reserve(sample_count as usize);
}
if TrunBox::FLAG_SAMPLE_CTS & flags > 0 {
sample_cts.reserve(sample_count as usize);
}
for _ in 0..sample_count {
if TrunBox::FLAG_SAMPLE_DURATION & flags > 0 {
let duration = reader.read_u32::<BigEndian>()?;
sample_durations.push(duration);
}
if TrunBox::FLAG_SAMPLE_SIZE & flags > 0 {
let sample_size = reader.read_u32::<BigEndian>()?;
sample_sizes.push(sample_size);
}
if TrunBox::FLAG_SAMPLE_FLAGS & flags > 0 {
let sample_flag = reader.read_u32::<BigEndian>()?;
sample_flags.push(sample_flag);
}
if TrunBox::FLAG_SAMPLE_CTS & flags > 0 {
let cts = reader.read_u32::<BigEndian>()?;
sample_cts.push(cts);
}
}
skip_bytes_to(reader, start + size)?;
Ok(TrunBox {
version,
flags,
sample_count,
data_offset,
first_sample_flags,
sample_durations,
sample_sizes,
sample_flags,
sample_cts,
})
}
}
impl<W: Write> WriteBox<&mut W> for TrunBox {
fn write_box(&self, writer: &mut W) -> Result<u64> {
let size = self.box_size();
BoxHeader::new(self.box_type(), size).write(writer)?;
write_box_header_ext(writer, self.version, self.flags)?;
writer.write_u32::<BigEndian>(self.sample_count)?;
if let Some(v) = self.data_offset {
writer.write_i32::<BigEndian>(v)?;
}
if let Some(v) = self.first_sample_flags {
writer.write_u32::<BigEndian>(v)?;
}
if self.sample_count != self.sample_sizes.len() as u32 {
return Err(Error::InvalidData("sample count out of sync"));
}
for i in 0..self.sample_count as usize {
if TrunBox::FLAG_SAMPLE_DURATION & self.flags > 0 {
writer.write_u32::<BigEndian>(self.sample_durations[i])?;
}
if TrunBox::FLAG_SAMPLE_SIZE & self.flags > 0 {
writer.write_u32::<BigEndian>(self.sample_sizes[i])?;
}
if TrunBox::FLAG_SAMPLE_FLAGS & self.flags > 0 {
writer.write_u32::<BigEndian>(self.sample_flags[i])?;
}
if TrunBox::FLAG_SAMPLE_CTS & self.flags > 0 {
writer.write_u32::<BigEndian>(self.sample_cts[i])?;
}
}
Ok(size)
}
}
#[cfg(test)]
mod tests {
use super::*;
use crate::mp4box::BoxHeader;
use std::io::Cursor;
#[test]
fn test_trun_same_size() {
let src_box = TrunBox {
version: 0,
flags: 0,
data_offset: None,
sample_count: 0,
sample_sizes: vec![],
sample_flags: vec![],
first_sample_flags: None,
sample_durations: vec![],
sample_cts: vec![],
};
let mut buf = Vec::new();
src_box.write_box(&mut buf).unwrap();
assert_eq!(buf.len(), src_box.box_size() as usize);
let mut reader = Cursor::new(&buf);
let header = BoxHeader::read(&mut reader).unwrap();
assert_eq!(header.name, BoxType::TrunBox);
assert_eq!(src_box.box_size(), header.size);
let dst_box = TrunBox::read_box(&mut reader, header.size).unwrap();
assert_eq!(src_box, dst_box);
}
#[test]
fn test_trun_many_sizes() {
let src_box = TrunBox {
version: 0,
flags: TrunBox::FLAG_SAMPLE_DURATION
| TrunBox::FLAG_SAMPLE_SIZE
| TrunBox::FLAG_SAMPLE_FLAGS
| TrunBox::FLAG_SAMPLE_CTS,
data_offset: None,
sample_count: 9,
sample_sizes: vec![1165, 11, 11, 8545, 10126, 10866, 9643, 9351, 7730],
sample_flags: vec![1165, 11, 11, 8545, 10126, 10866, 9643, 9351, 7730],
first_sample_flags: None,
sample_durations: vec![1165, 11, 11, 8545, 10126, 10866, 9643, 9351, 7730],
sample_cts: vec![1165, 11, 11, 8545, 10126, 10866, 9643, 9351, 7730],
};
let mut buf = Vec::new();
src_box.write_box(&mut buf).unwrap();
assert_eq!(buf.len(), src_box.box_size() as usize);
let mut reader = Cursor::new(&buf);
let header = BoxHeader::read(&mut reader).unwrap();
assert_eq!(header.name, BoxType::TrunBox);
assert_eq!(src_box.box_size(), header.size);
let dst_box = TrunBox::read_box(&mut reader, header.size).unwrap();
assert_eq!(src_box, dst_box);
}
}