From f9832f960fa8edcee0eb6866698cc5f9f25bd8f9 Mon Sep 17 00:00:00 2001 From: Emil Velikov Date: Sun, 16 Mar 2014 02:56:18 +0000 Subject: [PATCH] glx: drop obsolete _XUnlock_Mutex in __glXInitialize error path MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit With commit 1f1928db001(glx: Drop _Xglobal_lock while we create and initialize glx display) we've split the big _Xglobal_lock handling in a more fine grained manner. Unfortunatelly we forgot to drop the unlock_mutex on the error paths, leading to undefined behaviour as the mutex is already unlocked. Cc: Kristian Høgsberg Cc: "9.2 10.0 10.1" Signed-off-by: Emil Velikov Reviewed-by: Ian Romanick --- src/glx/glxext.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/glx/glxext.c b/src/glx/glxext.c index 0838cd19251..29317906d52 100644 --- a/src/glx/glxext.c +++ b/src/glx/glxext.c @@ -830,7 +830,6 @@ __glXInitialize(Display * dpy) dpyPriv->codes = XInitExtension(dpy, __glXExtensionName); if (!dpyPriv->codes) { free(dpyPriv); - _XUnlockMutex(_Xglobal_lock); return NULL; } @@ -846,7 +845,6 @@ __glXInitialize(Display * dpy) &dpyPriv->majorVersion, &dpyPriv->minorVersion) || (dpyPriv->majorVersion == 1 && dpyPriv->minorVersion < 1)) { free(dpyPriv); - _XUnlockMutex(_Xglobal_lock); return NULL; } @@ -911,7 +909,7 @@ __glXInitialize(Display * dpy) dpyPriv->next = glx_displays; glx_displays = dpyPriv; - _XUnlockMutex(_Xglobal_lock); + _XUnlockMutex(_Xglobal_lock); return dpyPriv; } -- 2.30.2