AddWindow: don't vtash reparenting

When identifying windows via XQueryTree(), ensure that both children and nchildren are considered. Fixes #785 Signed-off-by: Thomas Adam <thomas@fvwm.org>
fvwmorg · Dec 25, 2022 · 09d6b88 · 09d6b88
1 parent ad8e4a0
commit 09d6b88
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/fvwm/add_window.c b/fvwm/add_window.c
@@ -2599,12 +2599,12 @@ FvwmWindow *AddWindow(
 	}
 	{
 		unsigned int nchildren;
-		Window parent, *children;
+		Window parent, *children = NULL;
 		Status rc;
 
 		rc = XQueryTree(
 			dpy, w, &JunkRoot, &parent, &children, &nchildren);
-		if (nchildren > 0)
+		if (nchildren > 0 && children != NULL)
 		{
 			XFree(children);
 		}