Bugfix: Don’t free xinerama-reply if it is NULL, implement FREE()-macro (Thanks Igor)

[i3/i3] / src / xinerama.c

diff --git a/src/xinerama.c b/src/xinerama.c
index 8d00eb4cddff02205d7f8a9fee4c6161d5e9fc24..ac820bd8d14bb999b365e2ba8964257918f0bdf2 100644 (file)
--- a/src/xinerama.c
+++ b/src/xinerama.c
@@ -206,14 +206,14 @@ void initialize_xinerama(xcb_connection_t *conn) {
         xcb_xinerama_is_active_reply_t *reply;
         reply = xcb_xinerama_is_active_reply(conn, xcb_xinerama_is_active(conn), NULL);
 
-        if (!reply->state) {
+        if (reply == NULL || !reply->state) {
                 LOG("Xinerama is not active (in your X-Server), disabling.\n");
-                free(reply);
+                FREE(reply);
                 disable_xinerama(conn);
                 return;
         }
 
-        free(reply);
+        FREE(reply);
 
         query_screens(conn, virtual_screens);