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

diff --git a/i3bar/src/config.c b/i3bar/src/config.c
index 1f0c2a8eb2b3ec2439d3e65f8548a0ce7fa6de71..809b1ab4cfbec09840f919f3ae0bf42ab2e1efac 100644 (file)
--- a/i3bar/src/config.c
+++ b/i3bar/src/config.c
@@ -248,8 +248,7 @@ void parse_config_json(char *json) {
 void free_colors(struct xcb_color_strings_t *colors) {
 #define FREE_COLOR(x)        \
     do {                     \
-        if (colors->x)       \
-            free(colors->x); \
+        FREE(colors->x);     \
     } while (0)
     FREE_COLOR(bar_fg);
     FREE_COLOR(bar_bg);