desktop/window: avoid uaf on instant removal of a window

ref https://github.com/hyprwm/Hyprland/discussions/12999

1 files changed, 5 insertions, 5 deletions

diff --git a/src/desktop/view/Window.cpp b/src/desktop/view/Window.cpp
index 2559e0c4..7d5087e8 100644
--- a/src/desktop/view/Window.cpp
+++ b/src/desktop/view/Window.cpp
@@ -2648,15 +2648,15 @@ void CWindow::destroyWindow() {
 
     m_xdgSurface.reset();
 
-    if (!m_fadingOut) {
-        Log::logger->log(Log::DEBUG, "Unmapped {} removed instantly", m_self.lock());
-        g_pCompositor->removeWindowFromVectorSafe(m_self.lock()); // most likely X11 unmanaged or sumn
-    }
-
     m_listeners.unmap.reset();
     m_listeners.destroy.reset();
     m_listeners.map.reset();
     m_listeners.commit.reset();
+
+    if (!m_fadingOut) {
+        Log::logger->log(Log::DEBUG, "Unmapped {} removed instantly", m_self.lock());
+        g_pCompositor->removeWindowFromVectorSafe(m_self.lock()); // most likely X11 unmanaged or sumn
+    }
 }
 
 void CWindow::activateX11() {