Changeset 295345 in webkit

Timestamp:

Jun 7, 2022, 8:19:04 AM (4 years ago)

Author:

Adrian Perez de Castro

Message:

Merge r295336 - WebPermissionController::tryProcessingRequests should check for m_page
​https://bugs.webkit.org/show_bug.cgi?id=241344
rdar://94276618

Patch by Youenn Fablet <​youennf@gmail.com> on 2022-06-07
Reviewed by Eric Carlson.

• Source/WebKit/WebProcess/WebCoreSupport/WebPermissionController.cpp:

(WebKit::WebPermissionController::tryProcessingRequests):
Add a null check since async reply destructor might trigger completion handler execution.

Canonical link: ​https://commits.webkit.org/251355@main

File:

• releases/WebKitGTK/webkit-2.36/Source/WebKit/WebProcess/WebCoreSupport/WebPermissionController.cpp (modified) (1 diff)

releases/WebKitGTK/webkit-2.36/Source/WebKit/WebProcess/WebCoreSupport/WebPermissionController.cpp

@@ -89 +89 @@
 void WebPermissionController::tryProcessingRequests()
 {
-    if (m_requests.isEmpty())
+    if (m_requests.isEmpty() || !m_page)
         return;