xf86-video-intel: uxa/uxa-render.c

Chris Wilson ickle at kemper.freedesktop.org
Wed Jul 21 01:08:36 PDT 2010


 uxa/uxa-render.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit c57840b272ba88fddf22484929608431879b0fab
Author: Chris Wilson <chris at chris-wilson.co.uk>
Date:   Wed Jul 21 09:06:06 2010 +0100

    uxa: Check for failed pixmap allocation
    
    Fixes:
    
      Bug 29187 - crash in intel_drv
      https://bugs.freedesktop.org/show_bug.cgi?id=29187
    
    Backtrace:
    0: /usr/bin/X (xorg_backtrace+0x28) [0x466808]
    1: /usr/bin/X (0x400000+0x67c79) [0x467c79]
    2: /lib/libpthread.so.0 (0x7ff19b297000+0xef60) [0x7ff19b2a5f60]
    3: /usr/lib/xorg/modules/drivers/intel_drv.so (0x7ff197986000+0x34684) => uxa/uxa-render.c:841
    
    Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>

diff --git a/uxa/uxa-render.c b/uxa/uxa-render.c
index 9709a14..3194d23 100644
--- a/uxa/uxa-render.c
+++ b/uxa/uxa-render.c
@@ -835,8 +835,10 @@ uxa_acquire_drawable(ScreenPtr pScreen,
 			     PictureMatchFormat(pScreen, depth, pSrc->format),
 			     0, 0, serverClient, &error);
 	pScreen->DestroyPixmap(pPixmap);
-	ValidatePicture(pDst);
+	if (!pDst)
+		return 0;
 
+	ValidatePicture(pDst);
 done:
 	pDst->componentAlpha = pSrc->componentAlpha;
 	*out_x = 0;


More information about the xorg-commit mailing list