Make WordPress Core

Opened 5 years ago

Last modified 5 years ago

#22532 closed defect (bug)

It is assumed that a mime type has a minimum of two parts when splitted by "/" — at Version 1

Reported by: Chouby Owned by:
Milestone: 3.5 Priority: low
Severity: normal Version: 3.5
Component: Media Keywords: has-patch
Focuses: Cc:

Description (last modified by nacin)

While testing the version 3.5-beta3-22734, I got the following notice (due to an attachment created without media file):
PHP Notice: Undefined offset: 1 in /wp-includes/media.php on line 1242
The line is:

list( $type, $subtype ) = explode( '/', $attachment->post_mime_type );

See also: #22186

Change History (2)

#1 @nacin
5 years ago

  • Description modified (diff)
  • Milestone changed from Awaiting Review to 3.5
  • Priority changed from normal to low

Yeah, we should avoid making this assumption.

Note: See TracTickets for help on using tickets.