WordPress.org
Get WordPress

Make WordPress Core

Opened 8 weeks ago

Closed 4 weeks ago

#64669 closed defect (bug) (fixed)

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

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

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.
8 weeks 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
4 weeks ago

  • Milestone changed from Awaiting Review to 7.0

#3 @SergeyBiryukov
4 weeks ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

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.