Make WordPress Core

Opened 17 years ago

Closed 16 years ago

Last modified 16 years ago

#12276 closed defect (bug) (duplicate)

do_pings/do_all_pings emit E_WARNINGs on malformed URLs

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

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
17 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 UnassignedFuture Release

#3 @solarissmoke
16 years ago

  • Keywords needs-patch removed
  • Resolutionduplicate
  • Status newclosed

#4 @nacin
16 years ago

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