[PATCH] ramdac: Check sPriv != NULL in xf86CheckHWCursor()

Hans de Goede hdegoede at redhat.com
Tue Oct 25 08:11:16 UTC 2016


Hi,

On 25-10-16 00:25, Alex Goins wrote:
> xf86CheckHWCursor() would dereference sPriv without NULL checking it. If Option
> "SWCursor" is specified, sPriv == NULL. In this case we should assume that HW
> cursors are not supported.
>
> Signed-off-by: Alex Goins <agoins at nvidia.com>
> Reviewed-by: Andy Ritger <aritger at nvidia.com>

Patch looks good to me:

Reviewed-by: Hans de Goede <hdegoede at redhat.com>

Regards,

Hans


> ---
>  hw/xfree86/ramdac/xf86HWCurs.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/hw/xfree86/ramdac/xf86HWCurs.c b/hw/xfree86/ramdac/xf86HWCurs.c
> index da2b181..5e99526 100644
> --- a/hw/xfree86/ramdac/xf86HWCurs.c
> +++ b/hw/xfree86/ramdac/xf86HWCurs.c
> @@ -148,7 +148,8 @@ xf86CheckHWCursor(ScreenPtr pScreen, CursorPtr cursor, xf86CursorInfoPtr infoPtr
>              continue;
>
>          sPriv = dixLookupPrivate(&pSlave->devPrivates, xf86CursorScreenKey);
> -        if (!xf86ScreenCheckHWCursor(pSlave, cursor, sPriv->CursorInfoPtr))
> +        if (!sPriv ||
> +            !xf86ScreenCheckHWCursor(pSlave, cursor, sPriv->CursorInfoPtr))
>              return FALSE;
>      }
>      return TRUE;
>


More information about the xorg-devel mailing list