Attachment 23201 Details for Bug 20212 – Possible fix

[patch] Possible fix

0001-EXA-Re-hide-pixmap-data-pointer-after-ModifyPixmapH.patch (text/plain), 1.92 KB, created by Michel Dänzer on 2009-02-23 01:16:01 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Michel Dänzer

Created: 2009-02-23 01:16:01 UTC

Size: 1.92 KB

patch

obsolete

>From eb4d11e3c154ad4a26015885d493deff2da01f15 Mon Sep 17 00:00:00 2001
>From: =?utf-8?q?Michel=20D=C3=A4nzer?= <daenzer@vmware.com>
>Date: Mon, 23 Feb 2009 10:11:22 +0100
>Subject: EXA: Re-hide pixmap data pointer after ModifyPixmapHeader().
>
>---
> exa/exa.c |   22 ++++++++++------------
> 1 files changed, 10 insertions(+), 12 deletions(-)
>
>diff --git a/exa/exa.c b/exa/exa.c
>index 9329f10..fe13787 100644
>--- a/exa/exa.c
>+++ b/exa/exa.c
>@@ -318,11 +318,6 @@ exaCreatePixmap(ScreenPtr pScreen, int w, int h, int depth,
> 
>         datasize = h * paddedWidth;
> 
>-	/* Set this before driver hooks, to allow for !offscreen pixmaps.
>-	 * !offscreen pixmaps have a valid pointer at all times.
>-	 */
>-	pPixmap->devPrivate.ptr = NULL;
>-
>         pExaPixmap->driverPriv = pExaScr->info->CreatePixmap(pScreen, datasize, 0);
>         if (!pExaPixmap->driverPriv) {
> 	    swap(pExaScr, pScreen, DestroyPixmap);
>@@ -336,7 +331,6 @@ exaCreatePixmap(ScreenPtr pScreen, int w, int h, int depth,
>         pExaPixmap->score = EXA_PIXMAP_SCORE_PINNED;
>         pExaPixmap->fb_ptr = NULL;
>         pExaPixmap->pDamage = NULL;
>-        pExaPixmap->sys_ptr = pPixmap->devPrivate.ptr;
> 
>     } else {
>         pExaPixmap->driverPriv = NULL;
>@@ -434,14 +428,18 @@ exaModifyPixmapHeader(PixmapPtr pPixmap, int width, int height, int depth,
>     }
> 
> 
>-    if (pExaScr->info->ModifyPixmapHeader) {
>+    if (pExaScr->info->ModifyPixmapHeader)
> 	ret = pExaScr->info->ModifyPixmapHeader(pPixmap, width, height, depth,
> 						bitsPerPixel, devKind, pPixData);
>-	if (ret == TRUE)
>-	    return ret;
>-    }
>-    return pExaScr->SavedModifyPixmapHeader(pPixmap, width, height, depth,
>-					    bitsPerPixel, devKind, pPixData);
>+
>+    if (!ret)
>+	ret = pExaScr->SavedModifyPixmapHeader(pPixmap, width, height, depth,
>+					       bitsPerPixel, devKind, pPixData);
>+
>+    pExaPixmap->sys_ptr = pPixmap->devPrivate.ptr;
>+    pPixmap->devPrivate.ptr = NULL;
>+
>+    return ret;
> }
> 
> /**
>-- 
>1.6.2.rc1
>

Actions: View | Splinter Review

Attachments on bug 20212: 23112 | 23201 | 23205 | 23206 | 23207

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.