Ralovich, Kristóf wrote:
> This patch is at
> http://cgit.freedesktop.org/~tade/mesa/commit/?h=xcb-integration&id=fe0c68634ade8dafda2e8decaab077e50fea99e1
> .
> 
> 
> Kristof
> 
> 
>>From fe0c68634ade8dafda2e8decaab077e50fea99e1 Mon Sep 17 00:00:00 2001
> From: =?utf-8?q?Ralovich,=20Krist=C3=B3f?= <[EMAIL PROTECTED]>
> Date: Tue, 19 Aug 2008 13:31:45 +0200
> Subject: [PATCH] Free leaked psc->driScreen struct.
> 
> ---
>  src/glx/x11/glxext.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/src/glx/x11/glxext.c b/src/glx/x11/glxext.c
> index 7c75d01..edc7da2 100644
> --- a/src/glx/x11/glxext.c
> +++ b/src/glx/x11/glxext.c
> @@ -175,9 +175,12 @@ static void FreeScreenConfigs(__GLXdisplayPrivate *priv)
>           psc->driScreen->destroyScreen(psc);
>           __glxHashDestroy(psc->drawHash);
>        }
> +      XFree(psc->driScreen);
> +      psc->driScreen = NULL;
>  #endif
>     }
>     XFree((char*) priv->screenConfigs);
> +   priv->screenConfigs = NULL;
>  }
> 
>  /*
> --
> 1.5.6.3

OK, I moved the XFree(psc->driScreen) call inside the conditional that 
test psc->driScreen.

-Brian


-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Mesa3d-dev mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to