Unlike free(3), XFree(3) can't be passed a NULL pointer.

This commit is contained in:
okan 2014-02-02 16:13:50 +00:00
parent 0a71742af4
commit 5f9c7a21c7

View File

@ -65,7 +65,6 @@ screen_init(int which)
/* Deal with existing clients. */ /* Deal with existing clients. */
XQueryTree(X_Dpy, sc->rootwin, &w0, &w1, &wins, &nwins); XQueryTree(X_Dpy, sc->rootwin, &w0, &w1, &wins, &nwins);
for (i = 0; i < nwins; i++) { for (i = 0; i < nwins; i++) {
XGetWindowAttributes(X_Dpy, wins[i], &winattr); XGetWindowAttributes(X_Dpy, wins[i], &winattr);
if (winattr.override_redirect || if (winattr.override_redirect ||
@ -73,6 +72,7 @@ screen_init(int which)
continue; continue;
(void)client_init(wins[i], sc, winattr.map_state != IsUnmapped); (void)client_init(wins[i], sc, winattr.map_state != IsUnmapped);
} }
if (wins)
XFree(wins); XFree(wins);
screen_updatestackingorder(sc); screen_updatestackingorder(sc);