Index: WebCore/ChangeLog
authorkjk <kjk@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 7 Apr 2007 03:05:13 +0000 (03:05 +0000)
committerkjk <kjk@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 7 Apr 2007 03:05:13 +0000 (03:05 +0000)
commiteada0471eaa6c65082ede01445e2336a890fbc35
tree7094c2528eb5674dfa01549224d52e75385eca0e
parentdc7b9f49d8f73b5782f16792e24856fcdec212a9
Index: WebCore/ChangeLog
===================================================================
--- WebCore/ChangeLog (revision 20709)
+++ WebCore/ChangeLog (working copy)
@@ -1,3 +1,13 @@
+2007-04-04  Krzysztof Kowalczyk  <kkowalczyk@gmail.com>
+
+        Reviewed by darin.
+
+        Coverity fix. Coverity says:
+        "Event var_deref_model: Variable "(this)->clientptr" tracked as NULL was passed to a function that dereferences it."
+
+        * platform/image-decoders/gif/GIFImageReader.cpp:
+        (GIFImageReader::read):
+
 2007-04-04  Adele Peterson  <adele@apple.com>

         Reviewed by Oliver.
Index: WebCore/platform/image-decoders/gif/GIFImageReader.cpp
===================================================================
--- WebCore/platform/image-decoders/gif/GIFImageReader.cpp (revision 20709)
+++ WebCore/platform/image-decoders/gif/GIFImageReader.cpp (working copy)
@@ -760,7 +760,8 @@ bool GIFImageReader::read(const unsigned
       if (query == GIFImageDecoder::GIFSizeQuery || haltAtFrame == images_decoded) {
         // The decoder needs to stop.  Hand back the number of bytes we consumed from
         // buffer minus 9 (the amount we consumed to read the header).
-        clientptr->decodingHalted(len + 9);
+        if (clientptr)
+            clientptr->decodingHalted(len + 9);
         GETN(9, gif_image_header);
         return true;
       }

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@20776 268f45cc-cd09-0410-ab3c-d52691b4dbfc
WebCore/ChangeLog
WebCore/platform/image-decoders/gif/GIFImageReader.cpp