[mutter] window: check for possible loop in transients

[Olivier Fourdan <ofourdan src gnome org>]

commit 4e82a751fbb2733875b3fe37c556c78dd9937574
Author: Olivier Fourdan <ofourdan redhat com>
Date:   Tue Dec 15 16:55:29 2015 +0100

    window: check for possible loop in transients
    
    If a broken or naughty application tries set up its windows to create
    a loop in the transient relationship, mutter will hang, looping forever
    in meta_window_foreach_ancestor()
    
    To avoid looping infinitely at various point in the code, check for a
    possible loop when setting the transient relationship and deny the
    request to set a window transient for another if that would create a
    loop.
    
    Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=759299

 src/core/window.c |   21 +++++++++++++++++++++
 1 files changed, 21 insertions(+), 0 deletions(-)
---
diff --git a/src/core/window.c b/src/core/window.c
index 30e7b9a..45cfcf4 100644
--- a/src/core/window.c
+++ b/src/core/window.c
@@ -7388,10 +7388,31 @@ meta_window_set_gtk_dbus_properties (MetaWindow *window,
   g_object_thaw_notify (G_OBJECT (window));
 }
 
+static gboolean
+check_transient_for_loop (MetaWindow *window,
+                          MetaWindow *parent)
+{
+  while (parent)
+    {
+      if (parent->transient_for == window)
+          return TRUE;
+      parent = parent->transient_for;
+    }
+
+  return FALSE;
+}
+
 void
 meta_window_set_transient_for (MetaWindow *window,
                                MetaWindow *parent)
 {
+  if (check_transient_for_loop (window, parent))
+    {
+      meta_warning ("Setting %s transient for %s would create a loop.\n",
+                    window->desc, parent->desc);
+      return;
+    }
+
   if (meta_window_appears_focused (window) && window->transient_for != NULL)
     meta_window_propagate_focus_appearance (window, FALSE);