How to prevent having the "is not a valid image file" error when using GD function imagecreatefrom* in php?

Question

I am using imagecreatefromgif / imagecreatefromjpeg / imagecreatefromjpeg functions with uploaded images and sometimes get an error (e.g. Warning: imagecreatefromgif(): ... is not a valid GIF file) when the image is corrupted. I read lots of posts about this topic and couldn't find a "working" answer.

I did try some of the following to validate the image (as suggested on other posts) but none of them worked in ALL situations. If the image does not have the header info, the following works, but if the image has the header/mime type info and is corrupted, it doesn't work.

if( imagecreatefromjpeg($uploaded_image) !== false ) {
    // image is okay.
}

OR

try {
    $test = imagecreatefrompng($uploaded_image);
} catch (Exception $e) {
    echo 'Caught exception: ',  $e->getMessage(), "\n";
}

OR

using getimagesize($uploaded_image); // will return false or an error if image is not valid.

None of those situations worked with a corrupted image file that has the header info/mime type info.

I would like to prevent having those errors and if possible detect if the image is not corrupted and can be used without problems with any of the imagecreatefromgif / imagecreatefromjpeg / imagecreatefromjpeg functions. If corrupted, I would like to display an error message instead of running the funciton. I am looking for a solution in PHP.

Thank you for your help and suggestions.

php.net/manual/en/function.error-reporting.php

hakre
– hakre

2011-10-28 18:32:12 +00:00
Commented Oct 28, 2011 at 18:32 — hakre
– hakre, Commented Oct 28, 2011 at 18:32

Mikk · Accepted Answer · 2013-11-12 02:31:25Z

8

You can try

$gd = @imagecreatefromstring(file_get_contents($file_path));    
    if ($gd === false) {
        throw new Exception ('Image is corrupted');
    }

It should work with most image formats known to gd. Also if you need specific error message you can use error_get_last().

@ will suppress error messages, and imagecreatefromstring tries to open known image formats. If this fails, $gd will have value 'false' and no error messages are thrown.

Edit:

Please not that in this example @ operator also will suprpess any errors from file_get_contents function.

edited Nov 12, 2013 at 2:31

answered Oct 28, 2011 at 18:31

Mikk

2,2293 gold badges32 silver badges44 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Rapti Over a year ago

This is the best solution I could think of. All other solutions I found would analyze the URL for the file type, but this one does not need to and is way more reliable; especially if the file type is not visible from the URL. Also, it saves a lot of code. Thank you.

aesede Over a year ago

Nice one! I could process a lot of files with this but I have a situation (just 1 file) where script says "Image is corrupted" but I can see the image in the browser (Firefox 32.0), so I'm guessing that GD is not prepared to surpass small errors (again, image displays correctly in the browser, also in image viewers)

Collectives™ on Stack Overflow

How to prevent having the "is not a valid image file" error when using GD function imagecreatefrom* in php?

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related