Bug 73812 - [GTK] GIF image test crashes on 32- and 64-bit Release
Summary: [GTK] GIF image test crashes on 32- and 64-bit Release
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: noel gordon
URL:
Keywords:
Depends on: 72864
Blocks:
  Show dependency treegraph
 
Reported: 2011-12-05 02:59 PST by noel gordon
Modified: 2011-12-05 23:53 PST (History)
5 users (show)

See Also:


Attachments
Patch (1.92 KB, patch)
2011-12-05 15:20 PST, noel gordon
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description noel gordon 2011-12-05 02:59:52 PST
After http://trac.webkit.org/changeset/101975

  fast/backgrounds/animated-gif-as-background.html = CRASH
  fast/images/dont-crash-with-null-gif-frames.html = CRASH (on 64-bit only)
  fast/images/gif-loop-count.html = CRASH
Comment 1 noel gordon 2011-12-05 14:46:46 PST
% pwd
 third_party/WebKit/Source/WebCore/platform/image-decoders/gif
% ack m_data *

GIFImageDecoder.cpp
87:        reader.read((const unsigned char*)m_data->data(), m_data->size(), GIFFrameCountQuery, static_cast<unsigned>(-1));
192:    m_readOffset = m_data->size() - bytesLeft;
321:    if (!m_reader->read((const unsigned char*)m_data->data() + m_readOffset, m_data->size() - m_readOffset, query, haltAtFrame) && isAllDataReceived())
Comment 2 noel gordon 2011-12-05 14:47:51 PST
Hence, m_data->size() is used in the GIF decoder.
Comment 3 noel gordon 2011-12-05 15:14:26 PST
Ignore me: m_data->size() is the decoder input data.
Comment 4 noel gordon 2011-12-05 15:20:58 PST
Created attachment 117947 [details]
Patch
Comment 5 Adam Barth 2011-12-05 17:07:41 PST
Comment on attachment 117947 [details]
Patch

Ok.  Do you think this will fix the tests?
Comment 6 noel gordon 2011-12-05 17:14:54 PST
My belief is it won't fix them.  I don't have a GTK setup to test locally, so I'm prepared to try
this patch, and hence rule out whether the issue depends on the resize() or not.
Comment 7 WebKit Review Bot 2011-12-05 22:20:22 PST
Comment on attachment 117947 [details]
Patch

Clearing flags on attachment: 117947

Committed r102096: <http://trac.webkit.org/changeset/102096>
Comment 8 WebKit Review Bot 2011-12-05 22:20:27 PST
All reviewed patches have been landed.  Closing bug.
Comment 9 noel gordon 2011-12-05 23:53:48 PST
belief is one thing, results are another.

GTK Linux 32-bit Release @ r102096
 http://build.webkit.org/builders/GTK%20Linux%2032-bit%20Release/builds/19773
 2011-12-05 22:39:23,028 7601 worker.py:186 DEBUG worker/0 fast/backgrounds/animated-gif-as-background.html passed
 2011-12-05 22:41:49,861 7601 worker.py:186 DEBUG worker/0 fast/images/dont-crash-with-null-gif-frames.html passed
 2011-12-05 22:41:50,508 7601 worker.py:186 DEBUG worker/0 fast/images/gif-loop-count.html passed

GTK Linux 64-bit Release @ r102096
 http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release/builds/14197
 2011-12-05 22:26:17,985 7968 worker.py:186 DEBUG worker/17 fast/backgrounds/animated-gif-as-background.html passed
 2011-12-05 22:26:26,587 7931 worker.py:186 DEBUG worker/5 fast/images/dont-crash-with-null-gif-frames.html passed
 2011-12-05 22:26:27,177 7931 worker.py:186 DEBUG worker/5 fast/images/gif-loop-count.html passed

GTK Linux 64-bit Debug @ r102096
 http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Debug/builds/28212
 2011-12-05 22:37:17,192 12694 worker.py:186 DEBUG worker/3 fast/backgrounds/animated-gif-as-background.html passed
 2011-12-05 22:39:50,284 12685 worker.py:186 DEBUG worker/0 fast/images/dont-crash-with-null-gif-frames.html passed
 2011-12-05 22:39:51,022 12685 worker.py:186 DEBUG worker/0 fast/images/gif-loop-count.html passed

LGTM.