From 41b3d993baf5cbcff7045e63ad0d3d8aebf1dc67 Mon Sep 17 00:00:00 2001
From: stefan11111 <stefan11111github@gmail.com>
Date: Tue, 25 Aug 2026 23:50:53 +0300
Subject: [PATCH] glamor: Set `need_free_region` immediately after pixman
 region allocation

Otherwise, the region would leak when the pixmap is not in gpu memory

Signed-off-by: stefan11111 <stefan11111github@gmail.com>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/2289>
---
 glamor/glamor_compositerects.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/glamor/glamor_compositerects.c b/glamor/glamor_compositerects.c
index 0796fed41f..461a0633da 100644
--- a/glamor/glamor_compositerects.c
+++ b/glamor/glamor_compositerects.c
@@ -191,6 +191,8 @@ glamor_composite_rectangles(CARD8 op,
         return;
     }
 
+    need_free_region = TRUE;
+
     pixmap = glamor_get_drawable_pixmap(dst->pDrawable);
     priv = glamor_get_pixmap_private(pixmap);
 
@@ -201,8 +203,6 @@ glamor_composite_rectangles(CARD8 op,
         goto fallback;
     }
 
-    need_free_region = TRUE;
-
     DEBUGF("%s: drawable extents (%d, %d),(%d, %d) x %d\n",
            __FUNCTION__,
            RegionExtents(&region)->x1, RegionExtents(&region)->y1,