WindowServer: Tear down WSClientConnection asynchronously

If something goes wrong when trying to message a windowing client,
we now tear it down *on next event loop* instead of immediately.

This avoids having to deal with disconnected windows suddenly
disappearing from window lists while looping over them.

1 files changed, 3 insertions, 1 deletions

diff --git a/Servers/WindowServer/WSClientConnection.cpp b/Servers/WindowServer/WSClientConnection.cpp
index 114a98ed5d..0b02575837 100644
--- a/Servers/WindowServer/WSClientConnection.cpp
+++ b/Servers/WindowServer/WSClientConnection.cpp
@@ -81,7 +81,9 @@ WSClientConnection::~WSClientConnection()
 
 void WSClientConnection::die()
 {
-    s_connections->remove(client_id());
+    deferred_invoke([this](auto&) {
+        s_connections->remove(client_id());
+    });
 }
 
 void WSClientConnection::notify_about_new_screen_rect(const Rect& rect)