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

ispe required for every image item #74

Closed dukesook closed 8 months ago

dukesook commented 8 months ago

The sample image here fails the ComplianceWarden because items 2 & 3 don't have an ispe associated with them. However, these items are of type 'mime' and 'uri '. They're not images so they don't have a width or height.

This usually isn't an issue because most mime items don't have properties associated with them. However, the mime & uri items in the image above do in fact have at least 1 property associated with them. Looking in heif.cpp, the rule is only checked for items that have a property associated with them.

Is the file actually compliant? Are mime items considered to be "image items"?

rbouqueau commented 8 months ago

The link seems dead. Can you check it?

dukesook commented 8 months ago

Sorry about that, I updated the link to the correct url.

rbouqueau commented 8 months ago

Thanks for reporting. Should be fixed now.