Stop walking window tree if we get zero

It looks like some of the child windows are zero, which causes the next call to XQueryTree to blow up.
brndnmtthws · Feb 24, 2024 · 7059aa8 · 7059aa8
1 parent 56a81ef
commit 7059aa8
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/x11.cc b/src/x11.cc
@@ -1384,8 +1384,13 @@ inline Window last_descendant(Display *display, Window parent) {
   while (
       XQueryTree(display, current, &_ignored, &_ignored, &children, &count) &&
       count != 0) {
-    current = children[count - 1];
+    auto next = children[count - 1];
     XFree(children);
+    if (next != 0) {
+      current = next;
+    } else {
+      break;
+    }
   }
 
   return current;