[FFmpeg-devel] [PATCH] avformat/mov: ignore old infe box versions

[James Almer]

On 4/13/2024 10:11 AM, James Almer wrote:
> Some files with no image items have them, and were working prior to the recent
> HEIF parsing overhaul.
> Ignore such boxes instead, to recover the old behavior.
> 
> Fixes a regression since d9fed9df2a9e70c9375d3b2591db35c09303d369.
> 
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>   libavformat/mov.c | 13 ++++++++-----
>   1 file changed, 8 insertions(+), 5 deletions(-)
> 
> diff --git a/libavformat/mov.c b/libavformat/mov.c
> index 0c892b56c8..160e0b14fb 100644
> --- a/libavformat/mov.c
> +++ b/libavformat/mov.c
> @@ -8137,8 +8137,8 @@ static int mov_read_infe(MOVContext *c, AVIOContext *pb, MOVAtom atom, int idx)
>       size -= 4;
>   
>       if (version < 2) {
> -        av_log(c->fc, AV_LOG_ERROR, "infe: version < 2 not supported\n");
> -        return AVERROR_PATCHWELCOME;
> +        avpriv_report_missing_feature(c->fc, "infe version < 2");
> +        return 1;
>       }
>   
>       item_id = version > 2 ? avio_rb32(pb) : avio_rb16(pb);
> @@ -8209,6 +8209,8 @@ static int mov_read_iinf(MOVContext *c, AVIOContext *pb, MOVAtom atom)
>           ret = mov_read_infe(c, pb, infe, i);
>           if (ret < 0)
>               return ret;
> +        if (ret)
> +            return 0;
>       }
>   
>       c->found_iinf = 1;
> @@ -9508,14 +9510,15 @@ static int mov_read_header(AVFormatContext *s)
>               av_log(s, AV_LOG_ERROR, "error reading header\n");
>               return err;
>           }
> -    } while ((pb->seekable & AVIO_SEEKABLE_NORMAL) && !mov->found_moov && !mov->found_iloc && !mov->moov_retry++);
> -    if (!mov->found_moov && !mov->found_iloc) {
> +    } while ((pb->seekable & AVIO_SEEKABLE_NORMAL) &&
> +             !mov->found_moov && !mov->found_iloc && !mov->found_iinf && !mov->moov_retry++);
> +    if (!mov->found_moov && !mov->found_iloc && !mov->found_iinf) {
>           av_log(s, AV_LOG_ERROR, "moov atom not found\n");
>           return AVERROR_INVALIDDATA;
>       }
>       av_log(mov->fc, AV_LOG_TRACE, "on_parse_exit_offset=%"PRId64"\n", avio_tell(pb));
>   
> -    if (mov->found_iloc) {
> +    if (mov->found_iloc && mov->found_iinf) {
>           for (i = 0; i < mov->nb_heif_item; i++) {
>               HEIFItem *item = &mov->heif_item[i];
>               MOVStreamContext *sc;

Will apply.