fix: disable decoder strictness for clap and pixi properties (#57)

fixes #28
fdintino · Jul 8, 2024 · 506b7ad · 506b7ad
1 parent fb3a89c
commit 506b7ad
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 0 deletions.
diff --git a/src/pillow_avif/_avif.c b/src/pillow_avif/_avif.c
@@ -790,6 +790,14 @@ AvifDecoderNew(PyObject *self_, PyObject *args) {
         max_threads = default_max_threads;
     }
     self->decoder->maxThreads = max_threads;
+#endif
+#if AVIF_VERSION >= 90200
+    // Turn off libavif's 'clap' (clean aperture) property validation.
+    self->decoder->strictFlags &= ~AVIF_STRICT_CLAP_VALID;
+    // Allow the PixelInformationProperty ('pixi') to be missing in AV1 image
+    // items. libheif v1.11.0 and older does not add the 'pixi' item property to
+    // AV1 image items.
+    self->decoder->strictFlags &= ~AVIF_STRICT_PIXI_REQUIRED;
 #endif
     self->decoder->codecChoice = codec;
 

diff --git a/tests/images/chimera-missing-pixi.avif b/tests/images/chimera-missing-pixi.avif
diff --git a/tests/test_file_avif.py b/tests/test_file_avif.py
@@ -579,6 +579,11 @@ def test_p_mode_transparency(self):
 
         assert_image_similar(im_png.convert("RGBA"), Image.open(buf_out), 1)
 
+    def test_decoder_strict_flags(self):
+        # This would fail if full avif strictFlags were enabled
+        with Image.open("%s/tests/images/chimera-missing-pixi.avif" % CURR_DIR) as im:
+            assert im.size == (480, 270)
+
 
 class TestAvifAnimation:
     @contextmanager