Bugfix: Don’t crash when changing workspace when there is more than one column

diff --git a/src/workspace.c b/src/workspace.c
index 4663dfae0c7b1018c00a3fbc4ed5cf4927dbed45..36f807d241d54081e5f9b35717725b06d3c37e5b 100644 (file)
--- a/src/workspace.c
+++ b/src/workspace.c
@@ -44,21 +44,22 @@ Workspace *workspace_get(int number) {
                  *
                  * To distinguish between the first workspace and a NULL
                  * pointer, we store <workspace number> + 1. */
-                for (int c = 0; c < num_workspaces; c++)
+                for (int c = 0; c < num_workspaces; c++) {
                         FOR_TABLE(&(workspaces[c])) {
                                 Container *con = workspaces[c].table[cols][rows];
                                 if (con->workspace != NULL) {
                                         LOG("Handling con %p with pointer %p (num %d)\n", con, con->workspace, con->workspace->num);
                                         con->workspace = (Workspace*)(con->workspace->num + 1);
                                 }
-                                Client *current;
-                                SLIST_FOREACH(current, &(workspaces[c].focus_stack), focus_clients) {
-                                        if (current->workspace == NULL)
-                                                continue;
-                                        LOG("Handling client %p with pointer %p (num %d)\n", current, current->workspace, current->workspace->num);
-                                        current->workspace = (Workspace*)(current->workspace->num + 1);
-                                }
                         }
+                        Client *current;
+                        SLIST_FOREACH(current, &(workspaces[c].focus_stack), focus_clients) {
+                                if (current->workspace == NULL)
+                                        continue;
+                                LOG("Handling client %p with pointer %p (num %d)\n", current, current->workspace, current->workspace->num);
+                                current->workspace = (Workspace*)(current->workspace->num + 1);
+                        }
+                }
 
                 /* preserve c_ws */
                 c_ws = (Workspace*)(c_ws->num);
@@ -76,21 +77,22 @@ Workspace *workspace_get(int number) {
                  * by default, thus requiring re-rendering the layout. */
                 c_ws = workspace_get((int)c_ws);
 
-                for (int c = 0; c < old_num_workspaces; c++)
+                for (int c = 0; c < old_num_workspaces; c++) {
                         FOR_TABLE(&(workspaces[c])) {
                                 Container *con = workspaces[c].table[cols][rows];
                                 if (con->workspace != NULL) {
                                         LOG("Handling con %p with (num %d)\n", con, con->workspace);
                                         con->workspace = workspace_get((int)con->workspace - 1);
                                 }
-                                Client *current;
-                                SLIST_FOREACH(current, &(workspaces[c].focus_stack), focus_clients) {
-                                        if (current->workspace == NULL)
-                                                continue;
-                                        LOG("Handling client %p with (num %d)\n", current, current->workspace);
-                                        current->workspace = workspace_get((int)current->workspace - 1);
-                                }
                         }
+                        Client *current;
+                        SLIST_FOREACH(current, &(workspaces[c].focus_stack), focus_clients) {
+                                if (current->workspace == NULL)
+                                        continue;
+                                LOG("Handling client %p with (num %d)\n", current, current->workspace);
+                                current->workspace = workspace_get((int)current->workspace - 1);
+                        }
+                }
 
                 /* Initialize the new workspaces */
                 for (int c = old_num_workspaces; c < num_workspaces; c++) {