Ticket #17560: 17560.second.diff

wp-includes/post.php

@@ -2519 +2519 @@
                 $ping_status = get_option('default_ping_status');
 
         if ( isset($to_ping) )
-                $to_ping = preg_replace('|\s+|', "\n", $to_ping);
+                $to_ping = sanitize_trackback_urls( $to_ping );
         else
                 $to_ping = '';
 
@@ -3052 +3052 @@
 function get_to_ping($post_id) {
         global $wpdb;
         $to_ping = $wpdb->get_var( $wpdb->prepare( "SELECT to_ping FROM $wpdb->posts WHERE ID = %d", $post_id ));
-        $to_ping = trim($to_ping);
+        $to_ping = sanitize_trackback_urls( trim( $to_ping ) );
         $to_ping = preg_split('/\s/', $to_ping, -1, PREG_SPLIT_NO_EMPTY);
         $to_ping = apply_filters('get_to_ping',  $to_ping);
         return $to_ping;

wp-includes/formatting.php

@@ -2902 +2902 @@
         return apply_filters( 'sanitize_mime_type', $sani_mime_type, $mime_type );
 }
 
+/**
+ * Sanitize space or carriage return separated urls that are used to send trackbacks.
+ *
+ * @since 3.2.0
+ *
+ * @param string $to_ping Space or carriage return separated urls
+ * @return string Urls starting with the http or https protocol, separated by a carriage return.
+ */
+function sanitize_trackback_urls( $to_ping ) {
+    $urls_to_ping = preg_split('/\s/', $to_ping, -1, PREG_SPLIT_NO_EMPTY);
+    foreach( $urls_to_ping as $k => $url ) {
+        if ( !preg_match('#^https?://.#i', $url) )
+            unset($urls_to_ping[$k]);
+    }
+    $sani_to_ping = implode( "\n", $urls_to_ping );
+    return apply_filters( 'sanitize_trackback_urls', $sani_to_ping, $to_ping );
+}
+
 ?>