free config when reloading

1 files changed, 29 insertions, 12 deletions

diff --git a/sway/config.c b/sway/config.c
index 02805136..d2986a77 100644
--- a/sway/config.c
+++ b/sway/config.c
@@ -11,7 +11,7 @@
 #include "layout.h"
 #include "input_state.h"
 
-struct sway_config *config;
+struct sway_config *config = NULL;
 
 static bool file_exists(const char *path) {
 	return access(path, R_OK) != -1;
@@ -59,27 +59,41 @@ void free_mode(struct sway_mode *mode) {
 
 void free_config(struct sway_config *config) {
 	int i;
+	for (i = 0; i < config->symbols->length; ++i) {
+		struct sway_variable *var = config->symbols->items[i];
+		free(var->name);
+		free(var->value);
+		free(var);
+	}
+	list_free(config->symbols);
+
 	for (i = 0; i < config->modes->length; ++i) {
-		free_mode((struct sway_mode *)config->modes->items[i]);
+		free_mode(config->modes->items[i]);
 	}
-	free_flat_list(config->modes);
+	list_free(config->modes);
+
+	for (i = 0; i < config->cmd_queue->length; ++i) {
+		free(config->cmd_queue->items[i]);
+	}
+	list_free(config->cmd_queue);
+
 	for (i = 0; i < config->workspace_outputs->length; ++i) {
 		struct workspace_output *wso = config->workspace_outputs->items[i];
 		free(wso->output);
 		free(wso->workspace);
+		free(wso);
 	}
-	free_flat_list(config->workspace_outputs);
-	free_flat_list(config->cmd_queue);
-	for (i = 0; i < config->symbols->length; ++i) {
-		struct sway_variable *sym = config->symbols->items[i];
-		free(sym->name);
-		free(sym->value);
+	list_free(config->workspace_outputs);
+
+	for (i = 0; i < config->output_configs->length; ++i) {
+		struct output_config *oc = config->output_configs->items[i];
+		free(oc->name);
+		free(oc);
 	}
-	free_flat_list(config->symbols);
-	free_flat_list(config->output_configs);
+	list_free(config->output_configs);
+	free(config);
 }
 
-
 static char *get_config_path(void) {
 	char *config_path = NULL;
 	char *paths[3] = {getenv("HOME"), getenv("XDG_CONFIG_HOME"), ""};
@@ -244,6 +258,9 @@ _continue:
 		temp_config->reloading = false;
 		arrange_windows(&root_container, -1, -1);
 	}
+	if (config) {
+		free_config(config);
+	}
 	config = temp_config;
 
 	return success;