Avoid crash if ITP Debug mode is on, but ResourceLoadStatistics are not being used
authorbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Apr 2018 18:55:59 +0000 (18:55 +0000)
committerbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 18 Apr 2018 18:55:59 +0000 (18:55 +0000)
https://bugs.webkit.org/show_bug.cgi?id=184739
<rdar://problem/39287964>

Reviewed by David Kilzer.

If a user has the ResourceLoadStatistics logic turned off, but has the ITP Debug experimental
flag turned on, you can cause a crash.

This is because the WebsiteDataStore for the process doesn't bother creating a ResourceLoadStatisticsStore
if the statistics machinery is not running. The ITP debug flag was being blindly set without checking
if the statistics store exists or not, which can cause a crash.

* UIProcess/WebsiteData/WebsiteDataStore.cpp:
(WebKit::WebsiteDataStore::setResourceLoadStatisticsDebugMode): Check if the statistics store exists
before calling functions on it.

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

Source/WebKit/ChangeLog
Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp

index c9998d4..14c564b 100644 (file)
@@ -1,3 +1,22 @@
+2018-04-18  Brent Fulgham  <bfulgham@apple.com>
+
+        Avoid crash if ITP Debug mode is on, but ResourceLoadStatistics are not being used
+        https://bugs.webkit.org/show_bug.cgi?id=184739
+        <rdar://problem/39287964>
+
+        Reviewed by David Kilzer.
+
+        If a user has the ResourceLoadStatistics logic turned off, but has the ITP Debug experimental
+        flag turned on, you can cause a crash.
+
+        This is because the WebsiteDataStore for the process doesn't bother creating a ResourceLoadStatisticsStore
+        if the statistics machinery is not running. The ITP debug flag was being blindly set without checking
+        if the statistics store exists or not, which can cause a crash.
+
+        * UIProcess/WebsiteData/WebsiteDataStore.cpp:
+        (WebKit::WebsiteDataStore::setResourceLoadStatisticsDebugMode): Check if the statistics store exists
+        before calling functions on it.
+
 2018-04-18  Paul Knight  <pknight@apple.com>
 
         Add -[_WKInputDelegateDelegate willStartInputSession:] for clients that want to configure input session before assisting form node
index f993d32..2aa4661 100644 (file)
@@ -1435,7 +1435,8 @@ bool WebsiteDataStore::resourceLoadStatisticsDebugMode() const
 void WebsiteDataStore::setResourceLoadStatisticsDebugMode(bool enabled)
 {
     m_resourceLoadStatisticsDebugMode = enabled;
-    m_resourceLoadStatistics->setResourceLoadStatisticsDebugMode(enabled);
+    if (m_resourceLoadStatistics)
+        m_resourceLoadStatistics->setResourceLoadStatisticsDebugMode(enabled);
 }
 
 void WebsiteDataStore::enableResourceLoadStatisticsAndSetTestingCallback(Function<void (const String&)>&& callback)