Bugfix: Free property replies in their handlers

diff --git a/src/handlers.c b/src/handlers.c
index 2e3ad7aa0f80488cdf56e51ff49d4ac056eb5391..b20ef327aefbb2b96c2ab30f3d23d42be4503b9f 100644 (file)
--- a/src/handlers.c
+++ b/src/handlers.c
@@ -986,9 +986,8 @@ static int property_notify(uint8_t state, xcb_window_t window, xcb_atom_t atom)
         propr = xcb_get_property_reply(conn, cookie, 0);
     }
 
-    ret = handler->cb(NULL, conn, state, window, atom, propr);
-    FREE(propr);
-    return ret;
+    /* the handler will free() the reply */
+    return handler->cb(NULL, conn, state, window, atom, propr);
 }
 
 /*

diff --git a/src/window.c b/src/window.c
index b6aaae9e8f65e70539d3d5ebed1a5c3e3d694630..0449b1f53903df1b4714d95d9201837620d68145 100644 (file)
--- a/src/window.c
+++ b/src/window.c
@@ -37,6 +37,8 @@ void window_update_class(i3Window *win, xcb_get_property_reply_t *prop, bool bef
         return;
 
     run_assignments(win);
+
+    free(prop);
 }
 
 /*
@@ -80,6 +82,8 @@ void window_update_name(i3Window *win, xcb_get_property_reply_t *prop, bool befo
         return;
 
     run_assignments(win);
+
+    free(prop);
 }
 
 /*
@@ -121,6 +125,8 @@ void window_update_name_legacy(i3Window *win, xcb_get_property_reply_t *prop, bo
         return;
 
     run_assignments(win);
+
+    free(prop);
 }
 
 /*
@@ -140,6 +146,8 @@ void window_update_leader(i3Window *win, xcb_get_property_reply_t *prop) {
     DLOG("Client leader changed to %08x\n", *leader);
 
     win->leader = *leader;
+
+    free(prop);
 }
 
 /*
@@ -159,6 +167,8 @@ void window_update_transient_for(i3Window *win, xcb_get_property_reply_t *prop)
     DLOG("Transient for changed to %08x\n", transient_for);
 
     win->transient_for = transient_for;
+
+    free(prop);
 }
 
 /*
@@ -179,4 +189,6 @@ void window_update_strut_partial(i3Window *win, xcb_get_property_reply_t *prop)
          strut[0], strut[1], strut[2], strut[3]);
 
     win->reserved = (struct reservedpx){ strut[0], strut[1], strut[2], strut[3] };
+
+    free(prop);
 }