Not toggle floating on a CT_FLOATING_CON

See the issue #1875.
Forbid the command to toggle floating on a CT_FLOATING_CON to avoid the crash.

diff --git a/src/floating.c b/src/floating.c
index 67c3a1b81653d128c9c93ecf05ae73acaeb24e7b..d7a330678d1d2d8227e3037c9a690c3d30bbb861 100644 (file)
--- a/src/floating.c
+++ b/src/floating.c
@@ -369,6 +369,12 @@ void floating_disable(Con *con, bool automatic) {
  *
  */
 void toggle_floating_mode(Con *con, bool automatic) {
+    /* forbid the command to toggle floating on a CT_FLOATING_CON */
+    if (con->type == CT_FLOATING_CON) {
+        ELOG("Cannot toggle floating mode on con = %p because it is of type CT_FLOATING_CON.\n", con);
+        return;
+    }
+
     /* see if the client is already floating */
     if (con_is_floating(con)) {
         LOG("already floating, re-setting to tiling\n");