https://bugs.webkit.org/show_bug.cgi?id=56467
authorbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Mar 2011 16:38:28 +0000 (16:38 +0000)
committerbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 16 Mar 2011 16:38:28 +0000 (16:38 +0000)
IconDatabase-related crash seen in WK2 tests

Reviewed by Adam Roben.

* WebProcess/IconDatabase/WebIconDatabaseProxy.cpp:
(WebKit::WebIconDatabaseProxy::setIconDataForIconURL): The IconLoader might set a null data for an icon, so handle that case.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@81252 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/WebProcess/IconDatabase/WebIconDatabaseProxy.cpp

index 1c49d5b916f24816c3b1e0af259f9312c0c2a4f9..e74f93b20b318cd85cd16e9b97373c09fe7b5e45 100644 (file)
@@ -1,3 +1,13 @@
+2011-03-16  Brady Eidson  <beidson@apple.com>
+
+        Reviewed by Adam Roben.
+
+        https://bugs.webkit.org/show_bug.cgi?id=56467
+        IconDatabase-related crash seen in WK2 tests
+
+        * WebProcess/IconDatabase/WebIconDatabaseProxy.cpp:
+        (WebKit::WebIconDatabaseProxy::setIconDataForIconURL): The IconLoader might set a null data for an icon, so handle that case.
+
 2011-03-16  Benjamin Poulain  <benjamin.poulain@nokia.com>
 
         Reviewed by Kenneth Rohde Christiansen.
index fbdba148ffc4bfc07e2f37b6f49482c27877dd99..42a8eb46a56004c2317184bad66a0533aac8ee40 100644 (file)
@@ -109,7 +109,7 @@ void WebIconDatabaseProxy::setIconURLForPageURL(const String& iconURL, const Str
 
 void WebIconDatabaseProxy::setIconDataForIconURL(PassRefPtr<SharedBuffer> iconData, const String& iconURL)
 {
-    CoreIPC::DataReference data(reinterpret_cast<const uint8_t*>(iconData->data()), iconData->size());
+    CoreIPC::DataReference data(reinterpret_cast<const uint8_t*>(iconData ? iconData->data() : 0), iconData ? iconData->size() : 0);
     m_process->connection()->send(Messages::WebIconDatabase::SetIconDataForIconURL(data, iconURL), 0);
 }