Patch (ref bug #27572)

Keith Packard keithp at keithp.com
Fri May 7 22:20:00 PDT 2010


On Sat, 10 Apr 2010 19:16:40 +1000, telford at lnx-bsp.net wrote:

> +		if (! pDstPix) /* Usually implies display driver rejected width, height, depth, etc... */
> +		{
> +		    err = BadValue;
> +		    goto bail;
> +		}
> +

The only error you can legitimately return here is BadAlloc.

Otherwise, this looks like a fine patch, please be sure to fix it and
add a Signed-off-by: line so that I can merge it into the server.

-- 
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/20100507/467c63e5/attachment-0001.pgp>


More information about the xorg-devel mailing list