xserver: Branch 'master'

Adam Jackson ajax at kemper.freedesktop.org
Mon Feb 26 17:31:37 UTC 2018


 glx/glxext.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 22a3ffe68c9c498e0b6b74ebcf1404becda8e2b1
Author: Michel Dänzer <michel.daenzer at amd.com>
Date:   Mon Feb 26 17:30:57 2018 +0100

    glx: Don't pass NULL to glxGetClient
    
    These guards were dropped by the commit below, but it turns out they're
    needed. Fixes crash on VT switch.
    
    Fixes: d8ec33fe0542 ("glx: Use vnd layer for dispatch (v4)")
    Bugzilla: https://bugs.freedesktop.org/105233
    Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
    Reviewed-by: Adam Jackson <ajax at redhat.com>

diff --git a/glx/glxext.c b/glx/glxext.c
index 37416a4e4..f1355ce31 100644
--- a/glx/glxext.c
+++ b/glx/glxext.c
@@ -652,7 +652,7 @@ glxSuspendClients(void)
     int i;
 
     for (i = 1; i < currentMaxClients; i++) {
-        if (glxGetClient(clients[i])->client)
+        if (clients[i] && glxGetClient(clients[i])->client)
             IgnoreClient(clients[i]);
     }
 
@@ -668,7 +668,7 @@ glxResumeClients(void)
     glxBlockClients = FALSE;
 
     for (i = 1; i < currentMaxClients; i++) {
-        if (glxGetClient(clients[i])->client)
+        if (clients[i] && glxGetClient(clients[i])->client)
             AttendClient(clients[i]);
     }
 


More information about the xorg-commit mailing list