Changeset 254310 in webkit

Timestamp:

Jan 9, 2020, 3:47:34 PM (7 years ago)

Author:

wilander@apple.com

Message:

Check the existence of the optional m_sessionID before using it in WebProcess::setResourceLoadStatisticsEnabled()
​https://bugs.webkit.org/show_bug.cgi?id=206035
<rdar://problem/58455306>

Reviewed by Brent Fulgham.

No new tests.

• WebProcess/WebProcess.cpp:

(WebKit::WebProcess::setResourceLoadStatisticsEnabled):

Added a check that m_sessionID exists.

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• WebProcess/WebProcess.cpp (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2020-01-09  John Wilander  <wilander@apple.com>
+
+        Check the existence of the optional m_sessionID before using it in WebProcess::setResourceLoadStatisticsEnabled()
+        https://bugs.webkit.org/show_bug.cgi?id=206035
+        <rdar://problem/58455306>
+
+        Reviewed by Brent Fulgham.
+
+        No new tests.
+
+        * WebProcess/WebProcess.cpp:
+        (WebKit::WebProcess::setResourceLoadStatisticsEnabled):
+            Added a check that m_sessionID exists.
+
 2020-01-09  Tim Horton  <timothy_horton@apple.com>
 

trunk/Source/WebKit/WebProcess/WebProcess.cpp

@@ -1631 +1631 @@
 void WebProcess::setResourceLoadStatisticsEnabled(bool enabled)
 {
-    if (WebCore::DeprecatedGlobalSettings::resourceLoadStatisticsEnabled() == enabled || m_sessionID->isEphemeral())
+    if (WebCore::DeprecatedGlobalSettings::resourceLoadStatisticsEnabled() == enabled || (m_sessionID && m_sessionID->isEphemeral()))
         return;
     WebCore::DeprecatedGlobalSettings::setResourceLoadStatisticsEnabled(enabled);