gettextprop: check result of XGetTextProperty (undefined behaviour for XFree)
authorHiltjo Posthuma <hiltjo@codemadness.org>
Fri, 3 Nov 2017 15:36:32 +0000 (16:36 +0100)
committerHiltjo Posthuma <hiltjo@codemadness.org>
Fri, 3 Nov 2017 20:14:58 +0000 (21:14 +0100)
dwm.c

diff --git a/dwm.c b/dwm.c
index b0d99c0..650d19f 100644 (file)
--- a/dwm.c
+++ b/dwm.c
@@ -910,8 +910,7 @@ gettextprop(Window w, Atom atom, char *text, unsigned int size)
        if (!text || size == 0)
                return 0;
        text[0] = '\0';
-       XGetTextProperty(dpy, w, &name, atom);
-       if (!name.nitems)
+       if (!XGetTextProperty(dpy, w, &name, atom) || !name.nitems)
                return 0;
        if (name.encoding == XA_STRING)
                strncpy(text, (char *)name.value, size - 1);