commit c414c8b94cf9529071048d8de7bfeb00e05dd7c4
author Joe Drago <jdrago@netflix.com> Wed Sep 01 13:34:42 2021 -0700
committer Joe Drago <jdrago@netflix.com> Wed Sep 01 15:43:18 2021 -0700
tree ec7e3b42b41d52f51d8a24763e054fa475c9ed8f
parent bd1492e1812fff842c31c0f4da84d48f1349265c

NON-STANDARD: Allow alpha subimage items to not have an ispe property, infer ispe from color item

See: https://crbug.com/1245673

src/read.c

diff --git a/src/read.c b/src/read.c
index 667ff2f..bc5aa6f 100644
--- a/src/read.c
+++ b/src/read.c
@@ -3095,7 +3095,7 @@
                 avifDiagnosticsPrintf(data->diag, "Item ID [%u] size is too large [%ux%u]", item->id, item->width, item->height);
                 return AVIF_RESULT_BMFF_PARSE_FAILED;
             }
-        } else {
+        } else if (!item->auxForID) { // NON-STANDARD: Allow auxiliary images to not have an ispe property. See: https://crbug.com/1245673
             avifDiagnosticsPrintf(data->diag, "Item ID [%u] is missing a mandatory ispe property", item->id);
             return AVIF_RESULT_BMFF_PARSE_FAILED;
         }
@@ -3420,6 +3420,13 @@
         }
 
         if (alphaItem) {
+            if (!alphaItem->width && !alphaItem->height) {
+                // NON-STANDARD: Alpha subimage does not have an ispe property; adopt width/height from color item
+                // See: https://crbug.com/1245673
+                alphaItem->width = colorItem->width;
+                alphaItem->height = colorItem->height;
+            }
+
             if ((data->alphaGrid.rows > 0) && (data->alphaGrid.columns > 0)) {
                 if (!avifDecoderGenerateImageGridTiles(decoder, &data->alphaGrid, alphaItem, AVIF_TRUE)) {
                     return AVIF_RESULT_INVALID_IMAGE_GRID;