From 4bb89cdcdf56b58de51dd7dbc6af972f307abe63 Mon Sep 17 00:00:00 2001
From: "Enrico Weigelt, metux IT consult" <info@metux.net>
Date: Tue, 12 Aug 2025 18:51:31 +0200
Subject: [PATCH] damageext: canonical `walkScreen` variable on screen list
 iterations

When iterating screen lists, consistently use the same variable name
`walkScreen` for holding current screen pointer everywhere.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 damageext/damageext.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/damageext/damageext.c b/damageext/damageext.c
index 8a41185e01..aaf4fa0b46 100644
--- a/damageext/damageext.c
+++ b/damageext/damageext.c
@@ -647,12 +647,13 @@ PanoramiXDamageCreate(ClientPtr client, xDamageCreateReq *stuff)
     damage->ext = doDamageCreate(client, &rc, stuff);
     if (rc == Success && draw->type == XRT_WINDOW) {
         FOR_NSCREENS_FORWARD(i) {
+            ScreenPtr walkScreen = screenInfo.screens[i];
             DrawablePtr pDrawable;
             DamagePtr pDamage = DamageCreate(PanoramiXDamageReport,
                                              PanoramiXDamageExtDestroy,
                                              DamageReportRawRegion,
                                              FALSE,
-                                             screenInfo.screens[i],
+                                             walkScreen,
                                              damage);
             if (!pDamage) {
                 rc = BadAlloc;
@@ -716,8 +717,10 @@ DamageExtensionInit(void)
     ExtensionEntry *extEntry;
     int s;
 
-    for (s = 0; s < screenInfo.numScreens; s++)
-        DamageSetup(screenInfo.screens[s]);
+    for (s = 0; s < screenInfo.numScreens; s++) {
+        ScreenPtr walkScreen = screenInfo.screens[s];
+        DamageSetup(walkScreen);
+    }
 
     DamageExtType = CreateNewResourceType(FreeDamageExt, "DamageExt");
     if (!DamageExtType)