From b9632822b73b982669a2a59e02b2a98c2f7b47dd Mon Sep 17 00:00:00 2001
From: Simon Ser <contact@emersion.fr>
Date: Mon, 5 Sep 2022 09:18:47 +0200
Subject: [PATCH] protocol: add wl_compositor.error.bad_parent

This forbids loops in sub-surface trees.

Signed-off-by: Simon Ser <contact@emersion.fr>
---
 protocol/wayland.xml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/protocol/wayland.xml b/protocol/wayland.xml
index a633bcd..99732c5 100644
--- a/protocol/wayland.xml
+++ b/protocol/wayland.xml
@@ -2861,6 +2861,8 @@
     <enum name="error">
       <entry name="bad_surface" value="0"
 	     summary="the to-be sub-surface is invalid"/>
+      <entry name="bad_parent" value="1"
+	     summary="the to-be sub-surface parent is invalid"/>
     </enum>
 
     <request name="get_subsurface">
@@ -2878,6 +2880,10 @@
 	becomes visible on the next time the state of the parent surface is
 	applied.
 
+	The parent surface must not be one of the child surface's descendants,
+	and the parent must be different from the child surface, otherwise the
+	bad_parent protocol error is raised.
+
 	This request modifies the behaviour of wl_surface.commit request on
 	the sub-surface, see the documentation on wl_subsurface interface.
       </description>
-- 
2.7.4