Ticket #8497: 8497.diff

wp-includes/theme.php

@@ -180 +180 @@
         $theme_data = implode( '', file( $theme_file ) );
         $theme_data = str_replace ( '\r', '\n', $theme_data );
         if ( preg_match( '|Theme Name:(.*)$|mi', $theme_data, $theme_name ) )
-                $name = $theme = wp_kses( trim( $theme_name[1] ), $themes_allowed_tags );
+                $name = $theme = wp_kses( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $theme_name[1]) ), $themes_allowed_tags );
         else
                 $name = $theme = '';
 
         if ( preg_match( '|Theme URI:(.*)$|mi', $theme_data, $theme_uri ) )
-                $theme_uri = clean_url( trim( $theme_uri[1] ) );
+                $theme_uri = clean_url( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $theme_uri[1]) ) );
         else
                 $theme_uri = '';
 
         if ( preg_match( '|Description:(.*)$|mi', $theme_data, $description ) )
-                $description = wptexturize( wp_kses( trim( $description[1] ), $themes_allowed_tags ) );
+                $description = wptexturize( wp_kses( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $description[1]) ), $themes_allowed_tags ) );
         else
                 $description = '';
 
         if ( preg_match( '|Author URI:(.*)$|mi', $theme_data, $author_uri ) )
-                $author_uri = clean_url( trim( $author_uri[1]) );
+                $author_uri = clean_url( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $author_uri[1]) ) );
         else
                 $author_uri = '';
 
         if ( preg_match( '|Template:(.*)$|mi', $theme_data, $template ) )
-                $template = wp_kses( trim( $template[1] ), $themes_allowed_tags );
+                $template = wp_kses( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $template[1]) ), $themes_allowed_tags );
         else
                 $template = '';
 
         if ( preg_match( '|Version:(.*)|i', $theme_data, $version ) )
-                $version = wp_kses( trim( $version[1] ), $themes_allowed_tags );
+                $version = wp_kses( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $version[1]) ), $themes_allowed_tags );
         else
                 $version = '';
 
         if ( preg_match('|Status:(.*)|i', $theme_data, $status) )
-                $status = wp_kses( trim( $status[1] ), $themes_allowed_tags );
+                $status = wp_kses( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $status[1]) ), $themes_allowed_tags );
         else
                 $status = 'publish';
 
         if ( preg_match('|Tags:(.*)|i', $theme_data, $tags) )
-                $tags = array_map( 'trim', explode( ',', wp_kses( trim( $tags[1] ), array() ) ) );
+                $tags = array_map( 'trim', explode( ',', wp_kses( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $tags[1]) ), array() ) ) );
         else
                 $tags = array();
 
         if ( preg_match( '|Author:(.*)$|mi', $theme_data, $author_name ) ) {
                 if ( empty( $author_uri ) ) {
-                        $author = wp_kses( trim( $author_name[1] ), $themes_allowed_tags );
+                        $author = wp_kses( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $author_name[1]) ), $themes_allowed_tags );
                 } else {
-                        $author = sprintf( '<a href="%1$s" title="%2$s">%3$s</a>', $author_uri, __( 'Visit author homepage' ), wp_kses( trim( $author_name[1] ), $themes_allowed_tags ) );
+                        $author = sprintf( '<a href="%1$s" title="%2$s">%3$s</a>', $author_uri, __( 'Visit author homepage' ), wp_kses( trim( preg_replace("/\s*(?:\*\/|\?>).*/", '', $author_name[1]) ), $themes_allowed_tags ) );
                 }
         } else {
                 $author = __('Anonymous');

wp-admin/includes/plugin.php

@@ -86 +86 @@
 
         foreach ( array( 'name', 'uri', 'version', 'description', 'author_name', 'author_uri', 'text_domain', 'domain_path' ) as $field ) {
                 if ( !empty( ${$field} ) )
-                        ${$field} = trim(${$field}[1]);
+                        ${$field} = trim(preg_replace("/\s*(?:\*\/|\?>).*/", '', ${$field}[1]));
                 else
                         ${$field} = '';
         }

wp-admin/includes/file.php

@@ -52 +52 @@
         elseif ( file_exists( WP_CONTENT_DIR . $file ) && is_file( WP_CONTENT_DIR . $file ) ) {
                 $template_data = implode( '', file( WP_CONTENT_DIR . $file ) );
                 if ( preg_match( '|Template Name:(.*)$|mi', $template_data, $name ))
-                        return $name[1] . ' Page Template';
+                        return preg_replace("/\s*(?:\*\/|\?>).*/", '', $name[1]) . ' Page Template';
         }
 
         return basename( $file );

wp-admin/includes/theme.php

@@ -133 +133 @@
 
                         $name = '';
                         if ( preg_match( '|Template Name:(.*)$|mi', $template_data, $name ) )
-                                $name = $name[1];
+                                $name = preg_replace("/\s*(?:\*\/|\?>).*/", '', $name[1]);
 
                         if ( !empty( $name ) ) {
                                 $page_templates[trim( $name )] = basename( $template );