[Tickets #5315] Re: Horde_Image_gd code hits memory limit on large images

[bugs at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=5315
-----------------------------------------------------------------------
 Ticket             | 5315
 Updated By         | Michael Slusarz <slusarz at horde.org>
 Summary            | Horde_Image_gd code hits memory limit on large images
 Queue              | Horde Framework Packages
 Version            | HEAD
 Type               | Bug
-State              | Resolved
+State              | Feedback
 Priority           | 1. Low
 Owners             | 
-----------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2007-05-03 13:17) wrote:

> This could at least make it worse. I tweaked the code so that at 
> least the image data is not loaded in 
> IMP_MIME_Viewer_images::_getHordeImageObj() if not necessary. See if 
> this helps any.

Unfortunately this change will now potentially cause us to display an
invalid thumbnail image - or at least we will inform the user we are going
to display a thumbnail and then none will appear.  That's why we do the
check twice - the first time is to check if we can even attempt to load
the thumbnail.

For now, I have changed the code to return an empty string for the image
data but we may want to rethink the original patch.