WordPress.org

Make WordPress Core

#23733 closed defect (bug) (fixed)

Not all images have metadata, data should be checked first

Reported by: trusktr Owned by: markjaquith
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.5.1
Component: Media Keywords:
Focuses: Cc:

Description

I'm getting many errors like these. Is this a problem with a plugin, or with WordPress core?

The errors seems to be happening in core files, so I thought I'd post it here.

To see the actual errors, see the following site (at the time of this writing), in the right sidebar (scroll down):
http://creationofsociety.com

Attachments (3)

23733.diff (817 bytes) - added by wonderboymusic 13 months ago.
Fixes another instance
23733.2.diff (2.7 KB) - added by wonderboymusic 13 months ago.
More of them
23733.3.diff (8.9 KB) - added by wonderboymusic 13 months ago.

Download all attachments as: .zip

Change History (11)

comment:1 markoheijnen14 months ago

  • Keywords reporter-feedback added
  • Severity changed from major to normal

I need more information then this. Seems like the image metadata is corrupted but no clue why and what. Seems like a plugin is doing something wrong but hard to tell.

comment:2 trusktr14 months ago

The plugin that generates the sidebar widget where the errors show up is eshop, but the error doesn't point to any files from the eshop plugin.

comment:3 markoheijnen14 months ago

If you look at the code you see that the error isn't on that place. The data from wp_get_attachment_metadata() isn't how it should. Maybe a plugin does something wrong or you did a replace query on urls. However last one would most likely not show these errors.

comment:5 SergeyBiryukov13 months ago

  • Component changed from General to Media

wonderboymusic13 months ago

Fixes another instance

comment:6 wonderboymusic13 months ago

  • Keywords reporter-feedback removed
  • Milestone changed from Awaiting Review to 3.6
  • Summary changed from Warning: Illegal string offset 'width' in /wp-includes/media.php on line 440 to Not all images have metadata, data should be checked first

wonderboymusic13 months ago

More of them

wonderboymusic13 months ago

comment:8 markjaquith13 months ago

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

In 23873:

Sanity checks on image metadata to avoid warnings, etc.

fixes #23733. props wonderboymusic.

Note: See TracTickets for help on using tickets.