commit 3533b3355bb6e0e2c496aa8dfd5e650714a69e27
author Wan-Teh Chang <wtc@google.com> Fri Jan 02 18:06:06 2026 -0800
committer Wan-Teh Chang <wtc@google.com> Mon Jan 05 13:46:02 2026 -0800
tree 71c354ac424948a534f07e82ca6f16efd4eac3ba
parent 5277b7c90909b04732ce3a81ff098848f1e64f73

Use alphaItemCodecConfig only when valid

If alphaItemCodecConfigSize is equal to 0, alphaItemCodecConfig is
invalid (zero-initialized) and should not be used. This matches the
similar code for the gain map item.

Related to bug b:472695059.

src/read.c

diff --git a/src/read.c b/src/read.c
index 197c095..4cacf98 100644
--- a/src/read.c
+++ b/src/read.c
@@ -4436,13 +4436,17 @@
         AVIF_CHECKERR(avifMetaCreateProperty(meta, "skip"), AVIF_RESULT_OUT_OF_MEMORY); // Placeholder.
     }
 
-    if (hasAlpha) {
+    if (alphaItemCodecConfigSize != 0) {
         // Property with fixed index 6.
         avifProperty * alphaCodecConfigProp = avifMetaCreateProperty(meta, (const char *)codecConfigType);
         AVIF_CHECKERR(alphaCodecConfigProp, AVIF_RESULT_OUT_OF_MEMORY);
         alphaCodecConfigProp->u.av1C = alphaItemCodecConfig;
         AVIF_CHECKERR(avifDecoderItemAddProperty(alphaItem, alphaCodecConfigProp), AVIF_RESULT_OUT_OF_MEMORY);
+    } else {
+        AVIF_CHECKERR(avifMetaCreateProperty(meta, "skip"), AVIF_RESULT_OUT_OF_MEMORY); // Placeholder.
+    }
 
+    if (hasAlpha) {
         // Property with fixed index 7.
         alphaItem->auxForID = colorItem->id;
         colorItem->premByID = alphaIsPremultiplied;
@@ -4462,8 +4466,7 @@
         alphaPixiProp->u.pixi.planeDepths[0] = (uint8_t)bitDepth;
         AVIF_CHECKERR(avifDecoderItemAddProperty(alphaItem, alphaPixiProp), AVIF_RESULT_OUT_OF_MEMORY);
     } else {
-        // Placeholders 6, 7 and 8.
-        AVIF_CHECKERR(avifMetaCreateProperty(meta, "skip"), AVIF_RESULT_OUT_OF_MEMORY);
+        // Placeholders 7 and 8.
         AVIF_CHECKERR(avifMetaCreateProperty(meta, "skip"), AVIF_RESULT_OUT_OF_MEMORY);
         AVIF_CHECKERR(avifMetaCreateProperty(meta, "skip"), AVIF_RESULT_OUT_OF_MEMORY);
     }