Fixed some race conditions and dangling pointers when moving clients …

…+ toml parser (#432)

src/client.c

@@ -2197,16 +2197,9 @@ unmanage(Client *c, uint8_t destroyed)
     if(!c)
     {   return;
     }
-    Desktop *desk = c->desktop;
     const XCBWindow win = c->win;
 
-    if(desk->mon->bar == c)
-    {   desk->mon->bar = NULL;
-    }
     /* prevent dangling pointer here woops */
-    if(desk->sel == c)
-    {   desk->sel = NULL;
-    }
     if(!destroyed)
     {   
         /* TODO causes alot of errors for some reason even if its not "destroyed" */

src/desktop.c

@@ -170,6 +170,22 @@ detach(Client *c)
 void
 detachcompletely(Client *c)
 {
+    if(c)
+    {   
+        Desktop *desk = c->desktop;
+        if(desk)
+        {
+            Monitor *m = desk->mon;
+            if(m->bar == c)
+            {   
+                m->bar = NULL;
+                Debug0("Detaching bar? Potential memory leak");
+            }
+            if(desk->sel == c)
+            {   desk->sel = NULL;
+            }
+        }
+    }
     detach(c);
     detachstack(c);
     detachfocus(c);

src/main.c

@@ -28,6 +28,7 @@
 #include "main.h"
 #include "hashing.h"
 #include "getprop.h"
+#include "session.h"
 #include "keybinds.h"
 /* for HELP/DebugGING see under main() or the bottom */
 
@@ -240,6 +241,7 @@ cleanup(void)
 {
     /* save setting data. */
     USSave(&_cfg);
+    SessionSave();
     savesession();
     PropDestroy(_wm.handler);
     if(!_wm.dpy)