Changeset 45941 for branches/5.0/src/wp-admin/includes/ajax-actions.php

Timestamp:

09/04/2019 04:25:25 PM (6 years ago)

Author:

SergeyBiryukov

Message:

Escape the output in wp_ajax_upload_attachment().

Merges [45936] to the 5.0 branch.
Props whyisjake, sstoqnov.

Location:

branches/5.0

Files:

• . (modified) (1 prop)
• src/wp-admin/includes/ajax-actions.php (modified) (4 diffs)

branches/5.0/src/wp-admin/includes/ajax-actions.php

@@ -2069 +2069 @@
             'data'    => array(
                 'message'  => __( 'Sorry, you are not allowed to upload files.' ),
-                'filename' => $_FILES['async-upload']['name'],
+                'filename' => esc_html( $_FILES['async-upload']['name'] ),
             )
         ) );
@@ -2083 +2083 @@
                 'data'    => array(
                     'message'  => __( 'Sorry, you are not allowed to attach files to this post.' ),
-                    'filename' => $_FILES['async-upload']['name'],
+                    'filename' => esc_html( $_FILES['async-upload']['name'] ),
                 )
             ) );
@@ -2107 +2107 @@
                 'data'    => array(
                     'message'  => __( 'The uploaded file is not a valid image. Please try again.' ),
-                    'filename' => $_FILES['async-upload']['name'],
+                    'filename' => esc_html( $_FILES['async-upload']['name'] ),
                 )
             ) );
@@ -2122 +2122 @@
             'data'    => array(
                 'message'  => $attachment_id->get_error_message(),
-                'filename' => $_FILES['async-upload']['name'],
+                'filename' => esc_html( $_FILES['async-upload']['name'] ),
             )
         ) );