Bug 15097 - PNGImageDecoder.cpp throws away data in some cases
: PNGImageDecoder.cpp throws away data in some cases
Status: RESOLVED FIXED
: WebKit
Images
: 523.x (Safari 3)
: PC All
: P2 Normal
Assigned To:
:
:
:
:
  Show dependency treegraph
 
Reported: 2007-08-27 17:38 PST by
Modified: 2007-08-29 10:37 PST (History)


Attachments
patch v1 (2.73 KB, patch)
2007-08-27 17:41 PST, Peter Kasting
mjs: review+
Review Patch | Details | Formatted Diff | Diff


Note

You need to log in before you can comment on or make changes to this bug.


Description From 2007-08-27 17:38:46 PST
The code in PNGImageDecoder.cpp (not used for Safari Mac/Win, but used by Cairo/QT) can throw away data when consumers call any decoding-related functions on it before frameBufferAtIndex().

This is because the libpng code will only decode incoming data once (it remembers it's already decoded the existing data when asked to decode it again), and if no frame buffer is set up when it makes its callbacks, the relevant data is simply discarded.  And the frame buffer vector isn't resized to 1 frame (from 0) until frameBufferAtIndex() is called.

I don't see a good reason for this.  A single frame buffer entry takes up almost no memory before the underlying RGBA32Buffer is set up, so there really isn't a penalty for just sizing m_frameBufferCache to 1 in the constructor.  This also saves some conditionals in various places that no longer need to worry about whether this vector is empty.

Patch to fix coming shortly.
------- Comment #1 From 2007-08-27 17:41:19 PST -------
Created an attachment (id=16138) [details]
patch v1

With this patch, consumers can call any function on the PNGImageDecoder and trust that the image data won't be discarded.
------- Comment #2 From 2007-08-28 21:20:29 PST -------
(From update of attachment 16138 [details])
r=me
------- Comment #3 From 2007-08-29 10:37:38 PST -------
Landed in r25294.