Changes in trunk/wp-includes/kses.php [18208:17228]

File:

• trunk/wp-includes/kses.php (modified) (9 diffs)

trunk/wp-includes/kses.php

@@ -16 +16 @@
  * You should have received a copy of the GNU General Public License along
  * with this program; if not, write to the Free Software Foundation, Inc.,
- * 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
+ * 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA  or visit
  * http://www.gnu.org/licenses/gpl.html
  *
@@ -514 +514 @@
  *
  * There is currently only one kses WordPress hook and it is called here. All
- * parameters are passed to the hooks and expected to receive a string.
+ * parameters are passed to the hooks and expected to recieve a string.
  *
  * @since 1.0.0
@@ -555 +555 @@
         $pass_allowed_html = $allowed_html;
         $pass_allowed_protocols = $allowed_protocols;
-        return preg_replace_callback( '%(<!--.*?(-->|$))|(<[^>]*(>|$)|>)%', '_wp_kses_split_callback', $string );
+        return preg_replace_callback( '%((<!--.*?(-->|$))|(<[^>]*(>|$)|>))%', '_wp_kses_split_callback', $string );
 }
 
@@ -566 +566 @@
 function _wp_kses_split_callback( $match ) {
         global $pass_allowed_html, $pass_allowed_protocols;
-        return wp_kses_split2( $match[0], $pass_allowed_html, $pass_allowed_protocols );
+        return wp_kses_split2( $match[1], $pass_allowed_html, $pass_allowed_protocols );
 }
 
@@ -597 +597 @@
         # It matched a ">" character
 
-        if ( '<!--' == substr( $string, 0, 4 ) ) {
-                $string = str_replace( array('<!--', '-->'), '', $string );
-                while ( $string != ($newstring = wp_kses($string, $allowed_html, $allowed_protocols)) )
+        if (preg_match('%^<!--(.*?)(-->)?$%', $string, $matches)) {
+                $string = str_replace(array('<!--', '-->'), '', $matches[1]);
+                while ( $string != $newstring = wp_kses($string, $allowed_html, $allowed_protocols) )
                         $string = $newstring;
                 if ( $string == '' )
@@ -619 +619 @@
         $attrlist = $matches[3];
 
-        if ( ! isset($allowed_html[strtolower($elem)]) )
+        if (!@isset($allowed_html[strtolower($elem)]))
                 return '';
         # They are using a not allowed HTML element
 
         if ($slash != '')
-                return "</$elem>";
+                return "<$slash$elem>";
         # No attributes are allowed for closing elements
 
-        return wp_kses_attr( $elem, $attrlist, $allowed_html, $allowed_protocols );
+        return wp_kses_attr("$slash$elem", $attrlist, $allowed_html, $allowed_protocols);
 }
 
@@ -655 +655 @@
 
         # Are any attributes allowed at all for this element?
-        if ( ! isset($allowed_html[strtolower($element)]) || count($allowed_html[strtolower($element)]) == 0 )
+
+        if (@ count($allowed_html[strtolower($element)]) == 0)
                 return "<$element$xhtml_slash>";
 
         # Split it
+
         $attrarr = wp_kses_hair($attr, $allowed_protocols);
 
         # Go through $attrarr, and save the allowed attributes for this element
         # in $attr2
+
         $attr2 = '';
 
-        $allowed_attr = $allowed_html[strtolower($element)];
         foreach ($attrarr as $arreach) {
-                if ( ! isset( $allowed_attr[strtolower($arreach['name'])] ) )
+                if (!@ isset ($allowed_html[strtolower($element)][strtolower($arreach['name'])]))
                         continue; # the attribute is not allowed
 
-                $current = $allowed_attr[strtolower($arreach['name'])];
-                if ( $current == '' )
+                $current = $allowed_html[strtolower($element)][strtolower($arreach['name'])];
+                if ($current == '')
                         continue; # the attribute is not allowed
 
-                if ( ! is_array($current) ) {
+                if (!is_array($current))
                         $attr2 .= ' '.$arreach['whole'];
                 # there are no checks
 
-                } else {
+                else {
                         # there are some checks
                         $ok = true;
-                        foreach ($current as $currkey => $currval) {
-                                if ( ! wp_kses_check_attr_val($arreach['value'], $arreach['vless'], $currkey, $currval) ) {
+                        foreach ($current as $currkey => $currval)
+                                if (!wp_kses_check_attr_val($arreach['value'], $arreach['vless'], $currkey, $currval)) {
                                         $ok = false;
                                         break;
                                 }
-                        }
 
                         if ( strtolower($arreach['name']) == 'style' ) {
                                 $orig_value = $arreach['value'];
+
                                 $value = safecss_filter_attr($orig_value);
 
@@ -696 +698 @@
 
                                 $arreach['value'] = $value;
+
                                 $arreach['whole'] = str_replace($orig_value, $value, $arreach['whole']);
                         }
@@ -705 +708 @@
 
         # Remove any "<" or ">" characters
+
         $attr2 = preg_replace('/[<>]/', '', $attr2);