Crash when favicon is requested
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 15 Apr 2013 19:32:45 +0000 (19:32 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 15 Apr 2013 19:32:45 +0000 (19:32 +0000)
https://bugs.webkit.org/show_bug.cgi?id=114624

Patch by peavo@outlook.com <peavo@outlook.com> on 2013-04-15
Reviewed by Brent Fulgham.

* WebIconDatabase.cpp:
(WebIconDatabase::getOrCreateDefaultIconBitmap): Added NULL pointer check.

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

Source/WebKit/win/ChangeLog
Source/WebKit/win/WebIconDatabase.cpp

index 1bcd57b..6d26d1d 100644 (file)
@@ -1,3 +1,13 @@
+2013-04-15  peavo@outlook.com  <peavo@outlook.com>
+
+        Crash when favicon is requested
+        https://bugs.webkit.org/show_bug.cgi?id=114624
+
+        Reviewed by Brent Fulgham.
+
+        * WebIconDatabase.cpp:
+        (WebIconDatabase::getOrCreateDefaultIconBitmap): Added NULL pointer check.
+
 2013-04-12  Anders Carlsson  <andersca@apple.com>
 
         All tests crash on Windows
index 05a9490..6dc78e8 100644 (file)
@@ -307,7 +307,7 @@ HBITMAP WebIconDatabase::getOrCreateDefaultIconBitmap(LPSIZE size)
     result = createDIB(size);
 
     m_defaultIconMap.set(*size, result);
-    if (!iconDatabase().defaultIcon(*size)->getHBITMAPOfSize(result, size)) {
+    if (!iconDatabase().defaultIcon(*size) || !iconDatabase().defaultIcon(*size)->getHBITMAPOfSize(result, size)) {
         LOG_ERROR("Failed to draw Image to HBITMAP");
         return 0;
     }