gpac / ComplianceWarden

A pluggable compliance checker (ISOBMFF, HEIF/MIAF/AVIF, AV1 HDR10+)
https://gpac.github.io/ComplianceWarden-wasm/
Other
16 stars 7 forks source link

Incorrect validation of 'pasp' box size #11

Closed aklemets closed 4 years ago

aklemets commented 4 years ago

There seems to be something wrong with the test case for paragraph 12.1.3.2 in the ISOBMFF spec. It is requiring that the size of the 'pasp' box be 12 bytes instead of 16.

I don't have the latest ISOBMFF spec, but in the 5th version of the spec, the box consists of two 32-bit fields. That is 8 bytes. Plus 8 bytes for the Box header itself, equals 16 bytes.

rbouqueau commented 4 years ago

Fixed with #10