Use FREE macro for freeing colors

This macro will set the colors to NULL which avoids a double free on
reload when colors that were unconfigured become configured.

fixes #1299
This commit is contained in:
Tony Crisci 2014-07-01 23:17:02 -04:00 committed by Michael Stapelberg
parent b5f3679a66
commit 0095ca8770
1 changed files with 1 additions and 2 deletions

View File

@ -248,8 +248,7 @@ void parse_config_json(char *json) {
void free_colors(struct xcb_color_strings_t *colors) { void free_colors(struct xcb_color_strings_t *colors) {
#define FREE_COLOR(x) \ #define FREE_COLOR(x) \
do { \ do { \
if (colors->x) \ FREE(colors->x); \
free(colors->x); \
} while (0) } while (0)
FREE_COLOR(bar_fg); FREE_COLOR(bar_fg);
FREE_COLOR(bar_bg); FREE_COLOR(bar_bg);