diff options
author | wn_ <[email protected]> | 2021-02-22 18:03:36 +0000 |
---|---|---|
committer | wn_ <[email protected]> | 2021-02-22 18:03:36 +0000 |
commit | 7c966b69d5b06ea7c1bf418175db782d0e91e92d (patch) | |
tree | 3f32da2bd0c1d76011d82350f239060422db1dd7 /lib | |
parent | 42173386b39bed4b06c5ac6c2fc0da510673b354 (diff) |
Check whether data is parsable by 'imagecreatefromstring' in jimIcon.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/jimIcon.php | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/lib/jimIcon.php b/lib/jimIcon.php index f8e533f90..eaa768272 100644 --- a/lib/jimIcon.php +++ b/lib/jimIcon.php @@ -104,11 +104,11 @@ class jimIcon { } // See if we can parse it (might be PNG format here) - $i = @imagecreatefromstring($data); - - if ($i) { - imagesavealpha($i, true); - return $i; + if (self::has_parsable_image_type($data)) { + if ($i = @imagecreatefromstring($data)) { + imagesavealpha($i, true); + return $i; + } } // Must be a BMP. Parse it ourselves. @@ -267,5 +267,12 @@ class jimIcon { } return $img; } + + // Checks whether the data is a type parsable by imagecreatefromstring() + private function has_parsable_image_type($image_data) { + $size = getimagesizefromstring($image_data); + return $size && in_array($size[2], + [IMAGETYPE_JPEG, IMAGETYPE_PNG, IMAGETYPE_GIF, IMAGETYPE_BMP, IMAGETYPE_WBMP, IMAGETYPE_WEBP]); + } } ?> |