Как загрузить испорченную картинку через php?

Question

[Алексей]

Всем привет! При загрузке именно этого изображения - получаю вот такую ошибку php:

PHP Warning:  imagecreatefromjpeg(): gd-jpeg, libjpeg: recoverable error: Corrupt JPEG data: 34 extraneous bytes before marker 0xd1 in /home/........./SimpleImage.class.php on line 1243
PHP Warning:  imagecreatefromjpeg(): '/tmp/php7ZgA6S' is not a valid JPEG file in /home/......../SimpleImage.class.php on line 1243

На строке 1242-1244 вот этот код:

case 'image/jpeg':
   $this->image = imagecreatefromjpeg($this->filename);
   break;

На сколько я понимаю, картинка испорчена, тому свидетельство зеленая полоса чуть выше нижнего правого угла.
Но как решить эту проблему? Ведь например DLE также выбивает ошибку, что изображение испорчено, а вот тот же fastpic или toster спокойно её загружают...

Саму картинку прикрепляю в архиве https://yadi.sk/d/ycTz4UP_3JdR6V

UPD:
Как решение нашел вот такую настройку ini_set('gd.jpeg_ignore_warning', true);
В таком случае картинка прекрасно залетает и сохраняется, но безопасно ли это? Не залетит ли теперь с изображением какая-нить хрень?

Comments

[Nik Gubin]

Открыть с помощью графического редактора и пересохранить. Или загрузить в ВК, а потом выгрузить. ВК, если я не ошибаюсь, обрабатывает изображения и на выходе будет новый файл. Может "поломка" затрётся.

[Алексей]

Nik Gubin: Поломка-то затрется, но вопрос не в этом... как мне затирать такие поломки на своем сервере? Чтобы не объяснять юзерам, что нужно залить в ВК чтобы поломка затерлась...

[Nik Gubin]

Евгений: Это крайне редкий случай. Выбрасывайте сообщение, что фотография повреждена. А если там будет не полосочка забракована, а пол изображения? Смысл вам принимать такой контент

[Nik Gubin]

Евгений: если никто ранее не ответит, ближе к вечеру постараюсь выслать в ответе решение. Сейчас с телефона.

[Алексей]

Nik Gubin: А как сделать проверку на поврежденность изображения?

[Nik Gubin]

Евгений: обернуть вызов функции imagecreatefromjpeg в try {} catch() {}

case 'image/jpeg':
    try {
        $this->image = imagecreatefromjpeg($this->filename);
    } catch (\Exception $e) {
        // Этот блок будет вызван, если возникла какая-то ошибка при работе с картинкой
        // Можно присвоить $this->image = false, а ниже (под блоком) проверять, что он не false и работать дальше
    }
    break;

Update: но не исключено, что может вылететь ошибка при создании ресурса из картинки в png\gif\etc форматах, посему оберните в эту конструкцию весь свой switch, если там внутри нет ничего кроме создания ресурса (imagecreatefrom*).

[Алексей]

Nik Gubin: Спасибо, так и сделаю!
А зачем перед Exception обратный слеш?

[Nik Gubin]

Евгений: а это вам нужно про namespace прочесть.
Если в ваших скриптах такого нет, то слеш можно не ставить.
Обычно в начале документа есть что-то вроде namespace App\Foo\Bar;, а если нет ничего подобного, то и про пространство имён можете не думать и ставить без слеша.

Answer 1

[egor_nullptr]

Ключевой момент здесь libjpeg: recoverable error, поэтому с изображением можно работать, но warning лучше переделать в exception.

case 'image/jpeg':
   $prev_eh = set_error_handler(function ($no, $str, $file) {
        // GdException где-нибдуь выше определите как class GdException extends RuntimeException {};
        throw new GdException();
    }, E_WARNING);
    try {
        $this->image = imagecreatefromjpeg($this->filename);
    } catch (GdException $e) {
        // обработка битой картинки - например, через GraphicsMagick
    };
    set_error_handler($prev_eh);
    break;

А вообще, рекомендую перейти на GraphicsMagick или ImageMagick.

Answer 2

[4X_Pro]

Не очень понятна задача. Если нужно конкретно эту картинку загрузить, то можно открыть ее в каком-то графическом редакторе и сохранить заново (либо сконвертировать в PNG, а потом обратно в JPEG на своем компьютере), а потом загружать.
Если требуется убрать предупреждения при загрузке поврежденных картинок вообще, то нужно поставить @ перед imagecreatefromjpeg.