[PATCH] DGA: fix screen number check

Keith Packard keithp at keithp.com
Tue Nov 30 09:42:19 PST 2010


On Tue, 30 Nov 2010 17:23:10 +0100, Julien Cristau <jcristau at debian.org> wrote:
> screenInfo.numScreens is not a valid screen number, they go from 0 to
> numScreens - 1.
> 
> Signed-off-by: Julien Cristau <jcristau at debian.org>

Reviewed-by: Keith Packard <keithp at keithp.com>

It looks like this same error also exists in:

        hw/dmx/glxProxy/glxcmds.c
        hw/dmx/glxProxy/glxcmdswap.c
        hw/kdrive/ephyr/ephyrdriext.c

I found these with

        $ git grep screenInfo.numScreens | grep '[^-]>[^=]'

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20101130/a58029e6/attachment.pgp>


More information about the xorg-devel mailing list