#33220 [Asn->Csd]: GIF image with errors makes imagecreatefromgif() go into infinite loop
ID: 33220
Updated by: iliaa@php.net
Reported By: phpbugs at pureftpd dot org
-Status: Assigned
+Status: Closed
Bug Type: GD related
Operating System: *
PHP Version: 5CVS-2005-08-17
Assigned To: pajoye
New Comment:
Thank you for your bug report. This issue has already been fixed
in the latest released version of PHP, which you can download at
http://www.php.net/downloads.php
Previous Comments:
------------------------------------------------------------------------
[2005-11-06 23:21:35] sniper@php.net
Pierre, are there still open issues in this report? If not, close this.
Open separate reports for separate issues!
------------------------------------------------------------------------
[2005-09-24 16:42:01] pajoye@php.net
Patch applied in all active branches. I keep this bug as 'assigned' as
there is other cases which cause an infinite loop.
------------------------------------------------------------------------
[2005-09-24 12:23:44] nlopess@php.net
I've made a patch to fix this problem:
http://mega.ist.utl.pt/~ncpl/php_gd_patch.txt
The problem: GetDataBlock_() returns -1 on error, but most calls to
that function were checking only == 0, thus creating the infinite loop.
changing those to <=0 fixes the problems.
------------------------------------------------------------------------
[2005-06-02 15:11:16] phpbugs at pureftpd dot org
Description:
------------
The following (invalid) picture triggers an infinite loop
in gdImageCreateFromGif() with 100% CPU usage.
Tested on Linux and OpenBSD, PHP4 with GD 1 and PHP5 with
built-in GD.
Reproduce code:
---------------
<?php
imagecreatefromgif('kaboom-gd.gif');
?>
Get kaboom-gd.gif from ftp://ftp.00f.net/misc/kaboom-gd.gif
Expected result:
----------------
No 100% CPU usage.
Actual result:
--------------
100% CPU usage, infinite loop.
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=33220&edit=1
Thread (8 messages)