Changeset 40466 for branches/4.1/src/wp-admin/includes/media.php

Timestamp:

04/17/2017 01:28:48 PM (9 years ago)

Author:

swissspidy

Message:

Fix broken audio/video functions when sanitizing ID3 data

This fixes a bug where running wp_kses_post_deep() on all the ID3
tag data corrupted blob data.

See #40075, #40085.

Merges [40400] to the 4.1 branch.

Location:

branches/4.1

Files:

• . (modified) (1 prop)
• src/wp-admin/includes/media.php (modified) (3 diffs)

branches/4.1/src/wp-admin/includes/media.php

@@ -2894 +2894 @@
                         foreach ( $data[$version]['comments'] as $key => $list ) {
                                 if ( 'length' !== $key && ! empty( $list ) ) {
-                                        $metadata[$key] = reset( $list );
+                                        $metadata[$key] = wp_kses_post( reset( $list ) );
                                         // Fix bug in byte stream analysis.
                                         if ( 'terms_of_use' === $key && 0 === strpos( $metadata[$key], 'yright notice.' ) )
@@ -2978 +2978 @@
         wp_add_id3_tag_data( $metadata, $data );
 
-        $metadata = wp_kses_post_deep( $metadata );
-
         return $metadata;
 }
@@ -3019 +3017 @@
         wp_add_id3_tag_data( $metadata, $data );
 
-        $metadata = wp_kses_post_deep( $metadata );
-
         return $metadata;
 }