Ticket #10041: 10041.5.diff

wp-includes/taxonomy.php

@@ -1272 +1272 @@
         }
 
         if ( !empty($search) ) {
-                $search = like_escape($search);
+                $search = esc_sql(like_escape(stripslashes($search)));
                 $where .= " AND (t.name LIKE '%$search%')";
         }
 

wp-includes/bookmark.php

@@ -181 +181 @@
         }
 
         if ( ! empty($search) ) {
-                $search = like_escape($search);
+                $search = esc_sql(like_escape(stripslashes($search)));
                 $search = " AND ( (link_url LIKE '%$search%') OR (link_name LIKE '%$search%') OR (link_description LIKE '%$search%') ) ";
         }
 

wp-includes/formatting.php

@@ -2391 +2391 @@
  *
  * @since 2.5.0
  *
- * @param string $text The text to be escaped.
- * @return string text, safe for inclusion in LIKE query.
+ * @param string $text The text to be escaped (expected unslashed).
+ * @return string text, NOT safe for inclusion in LIKE query (needs an additional esc_sql()).
  */
 function like_escape($text) {
-        return str_replace(array("%", "_"), array("\\%", "\\_"), $text);
+        return addcslashes($text, '_%\\');
 }
 
 /**