Opened 2 months ago

Closed 8 weeks ago

#23733 closed defect (bug) (fixed)

Not all images have metadata, data should be checked first

Reported by: trusktr Owned by: markjaquith
Priority: normal Milestone: 3.6
Component: Media Version: 3.5.1
Severity: normal Keywords:
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 8 weeks ago.
Fixes another instance
23733.2.diff​ (2.7 KB) - added by wonderboymusic 8 weeks ago.
More of them
23733.3.diff​ (8.9 KB) - added by wonderboymusic 8 weeks ago.

Download all attachments as: .zip

Change History (11)

comment:1   markoheijnen — 2 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   trusktr — 2 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   markoheijnen — 2 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   SergeyBiryukov — 8 weeks ago

  • Component changed from General to Media

wonderboymusic — 8 weeks ago

Fixes another instance

comment:6   wonderboymusic — 8 weeks 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

wonderboymusic — 8 weeks ago

More of them

wonderboymusic — 8 weeks ago

comment:8   markjaquith — 8 weeks 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.