From ad0895ff0dc6e5080da5d5afff3f7b9d1f20a584 Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Wed, 3 Sep 2025 16:41:02 +0200 Subject: [PATCH] dbe: DbeExtensionInit() don't reuse walk variable differently The code is easier to understand when we don't reuse existing variables for entirely different things. Signed-off-by: Enrico Weigelt, metux IT consult --- dbe/dbe.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/dbe/dbe.c b/dbe/dbe.c index ea9e29d9dd..b684015c2f 100644 --- a/dbe/dbe.c +++ b/dbe/dbe.c @@ -1235,11 +1235,9 @@ DbeExtensionInit(void) */ for (int j = 0; j < i; j++) { - walkScreen = screenInfo.screens[j]; - free(dixLookupPrivate(&walkScreen->devPrivates, - dbeScreenPrivKey)); - dixSetPrivate(&walkScreen->devPrivates, - dbeScreenPrivKey, NULL); + ScreenPtr pScreen = screenInfo.screens[j]; + free(dixLookupPrivate(&pScreen->devPrivates, dbeScreenPrivKey)); + dixSetPrivate(&pScreen->devPrivates, dbeScreenPrivKey, NULL); } return; }