From 656a03e583a40b7ae92bcd142dc91e218cd45f78 Mon Sep 17 00:00:00 2001
From: David Rosca <david.rosca@amd.com>
Date: Mon, 9 Sep 2024 10:04:09 +0200
Subject: [PATCH] glx: Destroy base screen after deinitScreen

Instead of destroying it before deinitScreen to match the original order.

Fixes: 407a9094f70 ("glx: move base screen destroy to glx_screen_cleanup")

Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/31087>
---
 src/glx/glxext.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/glx/glxext.c b/src/glx/glxext.c
index ccf9b6cd564..d720d6959ee 100644
--- a/src/glx/glxext.c
+++ b/src/glx/glxext.c
@@ -241,6 +241,8 @@ FreeScreenConfigs(struct glx_display * priv)
 #if defined(GLX_DIRECT_RENDERING) && !defined(GLX_USE_APPLEGL)
       if (psc->driScreen.deinitScreen)
          psc->driScreen.deinitScreen(psc);
+      /* Free the direct rendering per screen data */
+      driDestroyScreen(psc->frontend_screen);
 #endif
       free(psc);
    }
@@ -743,8 +745,6 @@ glx_screen_cleanup(struct glx_screen *psc)
       driDestroyConfigs(psc->driver_configs);
       psc->driver_configs = NULL;
    }
-   /* Free the direct rendering per screen data */
-   driDestroyScreen(psc->frontend_screen);
 #endif
    free((char *) psc->serverGLXexts);
    free((char *) psc->serverGLXvendor);