From 7a6d2d41bd672f1fb21070210d9925ac38b90c5a Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Tue, 6 May 2025 17:02:39 +0200 Subject: [PATCH] Xext: shm: protect against possible NULL dereference MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit | ../Xext/shm.c: In function ‘ShmExtensionInit’: | ../Xext/shm.c:1534:29: warning: dereference of possibly-NULL ‘screen_priv’ [CWE-690] [-Wanalyzer-possible-null-dereference] | 1534 | if (!screen_priv->shmFuncs) | | ~~~~~~~~~~~^~~~~~~~~~ Signed-off-by: Enrico Weigelt, metux IT consult --- Xext/shm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Xext/shm.c b/Xext/shm.c index 446f9700a..e4cbf21dd 100644 --- a/Xext/shm.c +++ b/Xext/shm.c @@ -1536,6 +1536,8 @@ ShmExtensionInit(void) for (i = 0; i < screenInfo.numScreens; i++) { ShmScrPrivateRec *screen_priv = ShmInitScreenPriv(screenInfo.screens[i]); + if (!screen_priv) + continue; if (!screen_priv->shmFuncs) screen_priv->shmFuncs = &miFuncs; if (!screen_priv->shmFuncs->CreatePixmap)