Also check the type "skip" for a FreeSpaceBox

The type of a FreeSpaceBox can be "free" or "skip". A follow-up to AOMediaCodec#2398.
wantehchang · Aug 22, 2024 · a357f29 · a357f29
1 parent 9be2966
commit a357f29
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/read.c b/src/read.c
@@ -4153,7 +4153,8 @@ static avifResult avifParse(avifDecoder * decoder)
         }
 #endif
 
-        if (!isFtyp && (isNonSkippableVariableLengthBox || !memcmp(header.type, "free", 4) || !memcmp(header.type, "mdat", 4))) {
+        if (!isFtyp && (isNonSkippableVariableLengthBox || !memcmp(header.type, "free", 4) || !memcmp(header.type, "skip", 4) ||
+                        !memcmp(header.type, "mdat", 4))) {
             // Section 6.3.4 of ISO/IEC 14496-12:
             //   The FileTypeBox shall occur before any variable-length box (e.g. movie, free space, media data).
             AVIF_CHECKERR(ftypSeen, AVIF_RESULT_BMFF_PARSE_FAILED);