Changeset 21219 for trunk/wp-admin/includes/media.php

Timestamp:

07/05/2012 08:51:55 PM (12 years ago)

Author:

markjaquith

Message:

Better image extension regexes, so we don't just grab the "jpe" from "jpeg" files in media_sideload_image(). props sivel. fixes #16693

File:

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

trunk/wp-admin/includes/media.php

@@ -600 +600 @@
         // Set variables for storage
         // fix file filename for query strings
-        preg_match('/[^\?]+\.(jpg|JPG|jpe|JPE|jpeg|JPEG|gif|GIF|png|PNG)/', $file, $matches);
+        preg_match( '/[^\?]+\.(jpe?g|jpe|gif|png)\b/i', $file, $matches );
         $file_array['name'] = basename($matches[0]);
         $file_array['tmp_name'] = $tmp;