commit a117be888c88da76a400d3fa6f21f149374f8fc1
author Yannis Guyon <yguyon@google.com> Tue May 03 15:54:21 2022 +0200
committer Wan-Teh Chang <wtc@google.com> Tue May 03 23:24:48 2022 -0700
tree 32aa3ad6084acb51a6c30e1fdfd68cd5706d28a3
parent 570e918a32516a00dce5485d8022f75098a8ad6e

Avoid decoding partial tiles

Testing !data in avifDecoderDecodeTiles() is not enough because
samples gathered from merged extents may be partially available.
This behavior happens with incremental decoding of progressive
images, for example.

src/read.c

diff --git a/src/read.c b/src/read.c
index 70ed7fc..f367a6b 100644
--- a/src/read.c
+++ b/src/read.c
@@ -3776,7 +3776,7 @@
         avifTile * tile = &decoder->data->tiles.tile[firstTileIndex + tileIndex];
 
         const avifDecodeSample * sample = &tile->input->samples.sample[nextImageIndex];
-        if (!sample->data.data) {
+        if (sample->data.size < sample->size) {
             assert(decoder->allowIncremental);
             // Data is missing but there is no error yet. Output available pixel rows.
             return AVIF_RESULT_OK;