WordPress.org
Get WordPress

Make WordPress Core

Opened 16 years ago

Closed 15 years ago

Last modified 15 years ago

#12276 closed defect (bug) (duplicate)

do_pings/do_all_pings emit E_WARNINGs on malformed URLs

Reported by: rowanbeentje's profile rowanbeentje Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Pings/Trackbacks Keywords:
Focuses: Cc:

Description

When running WP_CRON, do_pings/do_all_pings causes E_WARNINGs and may abort on malformed URLs.

The backtrace looks like:
wp_cron.php: do_action_ref_array('do_pings')
wp_includes/plugin.php: do_all_pings()
wp_includes/comment.php: do_enclose()
wp_includes/functions.php: parse_url('​http://')

Obviously this is as a result of bad HTML, but then bad HTML is always bound to happen!

Both calls of parse_url within do_enclose should probably be silenced and the return values checked, returning if the URL can't be parsed?

Change History (4)

#1 @rowanbeentje
16 years ago

Oops, sorry, I could have formatted the backtrace properly! Hopefully you get the idea though...

#2 @nacin
16 years ago

  • Keywords needs-patch added
  • Milestone changed from Unassigned to Future Release

#3 @solarissmoke
15 years ago

  • Keywords needs-patch removed
  • Resolution set to duplicate
  • Status changed from new to closed

#11029

#4 @nacin
15 years ago

  • Milestone Future Release deleted
Note: See TracTickets for help on using tickets.