Fixed possible crash issue when getting an unmap before a focus in event.

The issue was caused by a client that got an unmap notify and had a FocusIn event
directly after in the event queue. The focus in event was handled by Fluxbox::revertFocus
before the WinClient.cc removed itself from the search window list.

1 files changed, 3 insertions, 4 deletions

diff --git a/src/WinClient.cc b/src/WinClient.cc
index 33d9fb4..4ef37ae 100644
--- a/src/WinClient.cc
+++ b/src/WinClient.cc
@@ -129,6 +129,9 @@ WinClient::~WinClient() {
     fbdbg<<__FILE__<<"(~"<<__FUNCTION__<<")[this="<<this<<"]"<<endl;
 
     FbTk::EventManager::instance()->remove(window());
+    Fluxbox *fluxbox = Fluxbox::instance();
+    if (window())
+        fluxbox->removeWindowSearch(window());
 
     clearStrut();
 
@@ -160,8 +163,6 @@ WinClient::~WinClient() {
     removeTransientFromWaitingList();
     s_transient_wait.erase(window());
 
-    Fluxbox *fluxbox = Fluxbox::instance();
-
     if (window_group != 0) {
         fluxbox->removeGroupSearch(window_group);
         window_group = 0;
@@ -170,8 +171,6 @@ WinClient::~WinClient() {
     if (m_mwm_hint != 0)
         XFree(m_mwm_hint);
 
-    if (window())
-        fluxbox->removeWindowSearch(window());
 }
 
 bool WinClient::acceptsFocus() const {