Re: [PATCH] glx: Refcnt the GLXDrawable to avoid use after free with multiple FreeResource

2011-02-02 Thread Chris Wilson
On Thu, 16 Dec 2010 15:55:42 +0100, Michel Dänzer wrote: > [ Dropping dri-devel list as this seems strictly an xserver issue ] > > On Fre, 2010-12-10 at 14:49 +0100, Michel Dänzer wrote: > > On Fre, 2010-12-10 at 13:38 +, Chris Wilson wrote: > > > Although there may be more than one resou

Re: [PATCH] glx: Refcnt the GLXDrawable to avoid use after free with multiple FreeResource

2010-12-16 Thread Michel Dänzer
On Don, 2010-12-16 at 11:54 -0500, Kristian Høgsberg wrote: > From: Chris Wilson > > Although there may be more than one resource handles pointing to the > Drawable, we only want to destroy it once and only reference the > resource which may have just been deleted on the first instance. > > v2:

[PATCH] glx: Refcnt the GLXDrawable to avoid use after free with multiple FreeResource

2010-12-16 Thread Kristian Høgsberg
From: Chris Wilson Although there may be more than one resource handles pointing to the Drawable, we only want to destroy it once and only reference the resource which may have just been deleted on the first instance. v2: Apply fixes and combine with another bug fix from Michel Dänzer, https

Re: [PATCH] glx: Refcnt the GLXDrawable to avoid use after free with multiple FreeResource

2010-12-16 Thread Michel Dänzer
[ Dropping dri-devel list as this seems strictly an xserver issue ] On Fre, 2010-12-10 at 14:49 +0100, Michel Dänzer wrote: > On Fre, 2010-12-10 at 13:38 +, Chris Wilson wrote: > > Although there may be more than one resource handles pointing to the > > Drawable, we only want to destroy it o

Re: [PATCH] glx: Refcnt the GLXDrawable to avoid use after free with multiple FreeResource

2010-12-10 Thread Michel Dänzer
On Fre, 2010-12-10 at 13:38 +, Chris Wilson wrote: > Although there may be more than one resource handles pointing to the > Drawable, we only want to destroy it once and only reference the > resource which may have just been deleted on the first instance. > > v2: Apply fixes and combine with

[PATCH] glx: Refcnt the GLXDrawable to avoid use after free with multiple FreeResource

2010-12-10 Thread Chris Wilson
Although there may be more than one resource handles pointing to the Drawable, we only want to destroy it once and only reference the resource which may have just been deleted on the first instance. v2: Apply fixes and combine with another bug fix from Michel Dänzer, https://bugs.freedesktop.o

Re: [PATCH] glx: Refcnt the GLXDrawable to avoid use after free with multiple FreeResource

2010-12-10 Thread Michel Dänzer
On Fre, 2010-12-10 at 12:43 +, Chris Wilson wrote: > Although there may be more than one resource handles pointing to the > Drawable, we only want to destroy it once and only reference the > resource which may have just been deleted on the first instance. > > Signed-off-by: Chris Wilson > Cc

[PATCH] glx: Refcnt the GLXDrawable to avoid use after free with multiple FreeResource

2010-12-10 Thread Chris Wilson
Although there may be more than one resource handles pointing to the Drawable, we only want to destroy it once and only reference the resource which may have just been deleted on the first instance. Signed-off-by: Chris Wilson Cc: Kristian Høgsberg Cc: Michel Dänzer --- glx/glxcmds.c |