Title: [278741] trunk/Source/WebKit
Revision
278741
Author
sihui_...@apple.com
Date
2021-06-10 16:55:52 -0700 (Thu, 10 Jun 2021)

Log Message

Regression(r278449): add null check for completionHandler in WebIDBServer::close
https://bugs.webkit.org/show_bug.cgi?id=226900
rdar://79160392

Reviewed by Chris Dumez.

* NetworkProcess/IndexedDB/WebIDBServer.cpp:
(WebKit::WebIDBServer::close):

Modified Paths

Diff

Modified: trunk/Source/WebKit/ChangeLog (278740 => 278741)


--- trunk/Source/WebKit/ChangeLog	2021-06-10 23:51:37 UTC (rev 278740)
+++ trunk/Source/WebKit/ChangeLog	2021-06-10 23:55:52 UTC (rev 278741)
@@ -1,3 +1,14 @@
+2021-06-10  Sihui Liu  <sihui_...@apple.com>
+
+        Regression(r278449): add null check for completionHandler in WebIDBServer::close
+        https://bugs.webkit.org/show_bug.cgi?id=226900
+        rdar://79160392
+
+        Reviewed by Chris Dumez.
+
+        * NetworkProcess/IndexedDB/WebIDBServer.cpp:
+        (WebKit::WebIDBServer::close):
+
 2021-06-10  Eric Carlson  <eric.carl...@apple.com>
 
         [Cocoa] Remote video layer should respect video gravity when resizing

Modified: trunk/Source/WebKit/NetworkProcess/IndexedDB/WebIDBServer.cpp (278740 => 278741)


--- trunk/Source/WebKit/NetworkProcess/IndexedDB/WebIDBServer.cpp	2021-06-10 23:51:37 UTC (rev 278740)
+++ trunk/Source/WebKit/NetworkProcess/IndexedDB/WebIDBServer.cpp	2021-06-10 23:55:52 UTC (rev 278741)
@@ -409,7 +409,7 @@
         connection.removeWorkQueueMessageReceiver(Messages::WebIDBServer::messageReceiverName());
 
     // Dispatch last task to clean up.
-   postTask([this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)]() mutable {
+    postTask([this, protectedThis = makeRef(*this), completionHandler = WTFMove(completionHandler)]() mutable {
         m_connectionMap.clear();
 
         {
@@ -418,7 +418,8 @@
         }
 
         postTaskReply([protectedThis = WTFMove(protectedThis), completionHandler = WTFMove(completionHandler)]() mutable {
-            completionHandler();
+            if (completionHandler)
+                completionHandler();
         });
     });
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to