aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLibravatar Drew DeVault <sir@cmpwn.com>2019-02-03 09:55:54 +0100
committerLibravatar GitHub <noreply@github.com>2019-02-03 09:55:54 +0100
commitbbfe13a24893bac6a5c41ac39115288d1d6e025f (patch)
tree1770223ab5d8fb7242452ab7886fdb214ad1aacd
parentMerge pull request #3554 from RedSoxFan/seat-current-alias (diff)
parentRevert "More frees." (diff)
downloadsway-bbfe13a24893bac6a5c41ac39115288d1d6e025f.tar.gz
sway-bbfe13a24893bac6a5c41ac39115288d1d6e025f.tar.zst
sway-bbfe13a24893bac6a5c41ac39115288d1d6e025f.zip
Merge pull request #3551 from c-edw/RevertDoubleFrees1.0-rc1
Revert double frees.
-rw-r--r--sway/config/bar.c1
-rw-r--r--sway/config/output.c1
-rw-r--r--sway/criteria.c7
-rw-r--r--swaybar/config.c1
-rw-r--r--swaybar/status_line.c2
5 files changed, 3 insertions, 9 deletions
diff --git a/sway/config/bar.c b/sway/config/bar.c
index 2e28fa1e..bafef307 100644
--- a/sway/config/bar.c
+++ b/sway/config/bar.c
@@ -46,7 +46,6 @@ void free_bar_config(struct bar_config *bar) {
46 free(bar->position); 46 free(bar->position);
47 free(bar->hidden_state); 47 free(bar->hidden_state);
48 free(bar->status_command); 48 free(bar->status_command);
49 free(bar->swaybar_command);
50 free(bar->font); 49 free(bar->font);
51 free(bar->separator_symbol); 50 free(bar->separator_symbol);
52 for (int i = 0; i < bar->bindings->length; i++) { 51 for (int i = 0; i < bar->bindings->length; i++) {
diff --git a/sway/config/output.c b/sway/config/output.c
index cc5f29db..970764b0 100644
--- a/sway/config/output.c
+++ b/sway/config/output.c
@@ -402,6 +402,5 @@ void free_output_config(struct output_config *oc) {
402 free(oc->name); 402 free(oc->name);
403 free(oc->background); 403 free(oc->background);
404 free(oc->background_option); 404 free(oc->background_option);
405 free(oc->background_fallback);
406 free(oc); 405 free(oc);
407} 406}
diff --git a/sway/criteria.c b/sway/criteria.c
index 59a923f9..48ef39df 100644
--- a/sway/criteria.c
+++ b/sway/criteria.c
@@ -35,19 +35,18 @@ bool criteria_is_empty(struct criteria *criteria) {
35} 35}
36 36
37void criteria_destroy(struct criteria *criteria) { 37void criteria_destroy(struct criteria *criteria) {
38 free(criteria->raw);
39 free(criteria->cmdlist);
40 free(criteria->target);
41 pcre_free(criteria->title); 38 pcre_free(criteria->title);
42 pcre_free(criteria->shell); 39 pcre_free(criteria->shell);
43 pcre_free(criteria->app_id); 40 pcre_free(criteria->app_id);
44 pcre_free(criteria->con_mark);
45#if HAVE_XWAYLAND 41#if HAVE_XWAYLAND
46 pcre_free(criteria->class); 42 pcre_free(criteria->class);
47 pcre_free(criteria->instance); 43 pcre_free(criteria->instance);
48 pcre_free(criteria->window_role); 44 pcre_free(criteria->window_role);
49#endif 45#endif
46 pcre_free(criteria->con_mark);
50 free(criteria->workspace); 47 free(criteria->workspace);
48 free(criteria->cmdlist);
49 free(criteria->raw);
51 free(criteria); 50 free(criteria);
52} 51}
53 52
diff --git a/swaybar/config.c b/swaybar/config.c
index b94fcfee..6e36573c 100644
--- a/swaybar/config.c
+++ b/swaybar/config.c
@@ -108,7 +108,6 @@ void free_config(struct swaybar_config *config) {
108 free(config->mode); 108 free(config->mode);
109 free(config->hidden_state); 109 free(config->hidden_state);
110 free(config->sep_symbol); 110 free(config->sep_symbol);
111 free(config->modifier);
112 for (int i = 0; i < config->bindings->length; i++) { 111 for (int i = 0; i < config->bindings->length; i++) {
113 struct swaybar_binding *binding = config->bindings->items[i]; 112 struct swaybar_binding *binding = config->bindings->items[i];
114 free_binding(binding); 113 free_binding(binding);
diff --git a/swaybar/status_line.c b/swaybar/status_line.c
index 17ada40c..342f981e 100644
--- a/swaybar/status_line.c
+++ b/swaybar/status_line.c
@@ -182,8 +182,6 @@ void status_line_free(struct status_line *status) {
182 } 182 }
183 json_tokener_free(status->tokener); 183 json_tokener_free(status->tokener);
184 } 184 }
185 free(status->read);
186 free(status->write);
187 free(status->buffer); 185 free(status->buffer);
188 free(status); 186 free(status);
189} 187}