[PATCH] Fix overlay detection when matching Xv adaptors across screens.

Adam Jackson ajax at nwnk.net
Wed Oct 7 12:34:44 PDT 2009


On Wed, 2009-10-07 at 10:42 -0700, Keith Packard wrote:
> Excerpts from Jamey Sharp's message of Wed Oct 07 09:07:03 -0700 2009:
> > Signed-off-by: Jamey Sharp <jamey at minilop.net>
> > ---
> > I don't know much about Xv or have any idea how to test this, but surely
> > this loop is broken without this patch?
> > 
> >  Xext/xvdisp.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/Xext/xvdisp.c b/Xext/xvdisp.c
> > index 04cd11a..5229916 100644
> > --- a/Xext/xvdisp.c
> > +++ b/Xext/xvdisp.c
> > @@ -1912,7 +1912,7 @@ void XineramifyXv(void)
> >          if((pAdapt->type & XvImageMask) && (pAdapt->nImages > 0)) {
> >            hasOverlay = FALSE;
> >                for(l = 0; l < pAdapt->nAttributes; l++) {
> > -             if(!strcmp(pAdapt->name, "XV_COLORKEY")) {
> > +             if(!strcmp(pAdapt->pAttributes[l].name, "XV_COLORKEY")) {
> >             hasOverlay = TRUE;
> >             break;
> >           }
> 
> Looks reasonable to me; anyone with an overlay who might test this patch?

I don't have a setup like that handy, but the code as is is clearly
wrong.

Acked-by: Adam Jackson <ajax at redhat.com>

- ajax
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
Url : http://lists.x.org/archives/xorg-devel/attachments/20091007/2c47ed75/attachment.pgp 


More information about the xorg-devel mailing list