Reviewed-by: Eamon Walsh
This should go as a bugfix on release branches; I'll take care of that after it
goes in. Thanks!
On 01/20/2011 11:01 PM, Adam Jackson wrote:
> M_DRAWABLE_PIXMAP is the lookup mask to dixLookupDrawable, and _not_ the
> type value in the drawable itself.
>
> Signed-off-by: Adam Jackson
> ---
> Xext/xselinux_ext.c |6 ++
> 1 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/Xext/xselinux_ext.c b/Xext/xselinux_ext.c
> index 93c1b59..374571c 100644
> --- a/Xext/xselinux_ext.c
> +++ b/Xext/xselinux_ext.c
> @@ -231,13 +231,11 @@ ProcSELinuxGetDrawableContext(ClientPtr client)
> REQUEST(SELinuxGetContextReq);
> REQUEST_SIZE_MATCH(SELinuxGetContextReq);
>
> -rc = dixLookupDrawable(&pDraw, stuff->id, client,
> -M_WINDOW | M_DRAWABLE_PIXMAP,
> -DixGetAttrAccess);
> +rc = dixLookupDrawable(&pDraw, stuff->id, client, 0, DixGetAttrAccess);
> if (rc != Success)
> return rc;
>
> -if (pDraw->type == M_DRAWABLE_PIXMAP)
> +if (pDraw->type == DRAWABLE_PIXMAP)
> privatePtr = &((PixmapPtr)pDraw)->devPrivates;
> else
> privatePtr = &((WindowPtr)pDraw)->devPrivates;
--
Eamon Walsh
National Security Agency
___
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel