Changeset 40462

Timestamp:

04/17/2017 01:14:01 PM (8 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.5 branch.

Location:

branches/4.5

Files:

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

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

@@ -2952 +2952 @@
             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.' ) )
@@ -3042 +3042 @@
     wp_add_id3_tag_data( $metadata, $data );
 
-    $metadata = wp_kses_post_deep( $metadata );
-
     return $metadata;
 }
@@ -3089 +3087 @@
     wp_add_id3_tag_data( $metadata, $data );
 
-    $metadata = wp_kses_post_deep( $metadata );
-
     return $metadata;
 }