From df3aa4922fd7e256169e541188b724f67ca948e1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Michel=20D=C3=A4nzer?= Date: Fri, 6 Nov 2020 10:14:19 +0100 Subject: [PATCH] xwayland: Make window_get_client_toplevel non-recursive Noticed while reading the code. Reviewed-by: Olivier Fourdan --- hw/xwayland/xwayland-window.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hw/xwayland/xwayland-window.c b/hw/xwayland/xwayland-window.c index d03790633..486149c4a 100644 --- a/hw/xwayland/xwayland-window.c +++ b/hw/xwayland/xwayland-window.c @@ -257,11 +257,11 @@ window_get_client_toplevel(WindowPtr window) * decoration/wrapper windows. In that case recurse by checking the client * of the first *and only* child of the decoration/wrapper window. */ - if (window_is_wm_window(window)) { - if (window->firstChild && window->firstChild == window->lastChild) - return window_get_client_toplevel(window->firstChild); - else + while (window_is_wm_window(window)) { + if (!window->firstChild || window->firstChild != window->lastChild) return NULL; /* Should never happen, skip resolution emulation */ + + window = window->firstChild; } return window;