Changeset 22521 for trunk/wp-admin/includes/ajax-actions.php

Timestamp:

11/10/2012 05:36:37 AM (12 years ago)

Author:

nacin

Message:

Don't allow non-image uploads for custom headers and backgrounds. props kovshenin. fixes #22149.

File:

• trunk/wp-admin/includes/ajax-actions.php (modified) (1 diff)

trunk/wp-admin/includes/ajax-actions.php

@@ -1610 +1610 @@
     $post_data = isset( $_REQUEST['post_data'] ) ? $_REQUEST['post_data'] : array();
 
+    // If the context is custom header or background, make sure the uploaded file is an image.
+    if ( isset( $post_data['context'] ) && in_array( $post_data['context'], array( 'custom-header', 'custom-background' ) ) ) {
+        $wp_filetype = wp_check_filetype_and_ext( $_FILES['async-upload']['tmp_name'], $_FILES['async-upload']['name'], false );
+        if ( ! wp_match_mime_types( 'image', $wp_filetype['type'] ) ) {
+            wp_send_json_error( array(
+                'message' => __( 'The uploaded file is not a valid image. Please try again.' ),
+                'filename' => $_FILES['async-upload']['name'],
+            ) );
+        }
+    }
+
     $attachment_id = media_handle_upload( 'async-upload', $post_id, $post_data );