[PATCH] xf86: return NULL for xf86CompatOutput if config->compat_output is -1

Keith Packard keithp at keithp.com
Mon Mar 18 11:31:59 PDT 2013


Aaron Plattner <aplattner at nvidia.com> writes:

> If there is no compat output, config->compat_output is -1 and xf86CompatOutput
> reads before the beginning of the outputs array.

Did you happen to review users of this function and make sure they all
handle NULL return values? The other 'obvious' return value would be to
just pick some output and hope it was reasonable, after all.

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


More information about the xorg-devel mailing list