Make WordPress Core

Opened 7 months ago

Closed 6 months ago

#64669 closed defect (bug) (fixed)

AVIF files with empty iref box fail parsing with older PHP versions

Reported by: yguyon Owned by: SergeyBiryukov
Priority: normal Milestone: 7.0
Component: Media Version:
Severity: minor Keywords: has-patch
Cc: Focuses:

Description

libavifinfo is used for PHP versions until 8.2 (1) to extract a few image features from AVIF files.

The current version of libavifinfo in WordPress rejects files with an empty 'iref' box. An empty 'iref' box is pointless but is allowed by the specification. See https://aomedia.g-issues.chromium.org/issues/466109759#comment3.

(1) https://github.com/WordPress/wordpress-develop/blob/5b7a3ad163b5b5aad890c1e7098d746bccf1012d/src/wp-includes/class-avif-info.php#L16

Change History (3)

This ticket was mentioned in PR #10972 on WordPress/wordpress-develop by @yguyon.


7 months ago
#1

  • Keywords has-patch added

Support AVIF files with an empty 'iref' box.

See the similar fix in PHP-src: https://github.com/php/php-src/pull/21250

#2 @SergeyBiryukov
6 months ago

  • Milestone Awaiting Review7.0

#3 @SergeyBiryukov
6 months ago

  • Owner set to SergeyBiryukov
  • Resolutionfixed
  • Status newclosed

In 62029:

Media: Correct parsing AVIF files with empty iref box on older PHP versions.

An empty iref box is pointless but is allowed by the specification. This commit imports an upstream fix from libavifinfo.

Follow-up to [57524], [58049].

Props yguyon.
Fixes #64669.

Note: See TracTickets for help on using tickets.