[wp-trac] [WordPress Trac] #12276: do_pings/do_all_pings emit E_WARNINGs on malformed URLs

WordPress Trac wp-trac at lists.automattic.com
Thu Feb 18 17:15:11 UTC 2010


#12276: do_pings/do_all_pings emit E_WARNINGs on malformed URLs
------------------------------+---------------------------------------------
 Reporter:  rowanbeentje      |       Owner:            
     Type:  defect (bug)      |      Status:  new       
 Priority:  normal            |   Milestone:  Unassigned
Component:  Pings/Trackbacks  |     Version:            
 Severity:  normal            |    Keywords:            
------------------------------+---------------------------------------------
 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?

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/12276>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list