[PATCH 2/3] dri2: free slave pixmap on app exit

2012-08-03 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com When the drawable disappears we need to free the prime master/slave combos. This fixes a leak after a prime app is run. Signed-off-by: Dave Airlie airl...@redhat.com --- hw/xfree86/dri2/dri2.c | 5 + 1 file changed, 5 insertions(+) diff --git

[PATCH 1/3] pixmap: have slave pixmap take a reference on master pixmap

2012-08-03 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com Since the free routines free the master pixmap then the slave, we should be taking a reference when we bind them together. Fixes a use-after-free when resizing a primed gears. Signed-off-by: Dave Airlie airl...@redhat.com --- dix/pixmap.c | 4 1 file

[PATCH 3/3] dri2: fix master pixmap free and reset pointer

2012-08-03 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com These are two minor changes, one to reset the pointer to NULL, after freeing the pixmaps, one to make sure we use the right API for the master pixmap, though I doubt it'll ever really matter. Signed-off-by: Dave Airlie airl...@redhat.com ---

Re: [PATCH 1/3] pixmap: have slave pixmap take a reference on master pixmap

2012-08-03 Thread Alex Deucher
On Fri, Aug 3, 2012 at 2:44 AM, Dave Airlie airl...@gmail.com wrote: From: Dave Airlie airl...@redhat.com Since the free routines free the master pixmap then the slave, we should be taking a reference when we bind them together. Fixes a use-after-free when resizing a primed gears.

Re: [PATCH] glx: Skip multisampled configs when matching pre-existing X visuals.

2012-08-03 Thread Chad Versace
On 08/01/2012 10:43 PM, Ian Romanick wrote: On 07/31/2012 03:15 PM, Paul Berry wrote: In __glXScreenInit() we generate the set of GLX visuals in two steps: first we match each pre-existing X visual with a corresponding FBConfig, then we generate a new X visual to correspond to all the

Re: [PATCH test/xts] Fix XSendEvent tests

2012-08-03 Thread Jeremy Huddleston Sequoia
For the series of 3 patches: Reviewed-by: Jeremy Huddleston Sequoia jerem...@apple.com On Aug 1, 2012, at 08:01, Peter Harris phar...@opentext.com wrote: XSendEvent of GenericEvent will return a Value error if the extension field isn't an extension that supports GenericEvent. Even before

[PULL] mainly hw/xwin fixes

2012-08-03 Thread Jon TURNEY
Hi Keith, Various fixes, mainly for hw/xwin. Please consider pulling into master. Thanks. The following changes since commit afa53fe7cffd430cf11f25ca818cb955a78c0c1c: Version 1.12.99.903 (1.13 RC3) (2012-07-25 14:29:48 -0700) are available in the git repository at: