[PATCH xfree86] Don't keep a pointer to a possibly freed cursor when changing screens, preventing a crash in xf86CursorEnableDisableFBAccess() trying to restore it.

Peter Hutterer peter.hutterer at who-t.net
Mon Apr 5 16:23:49 PDT 2010


On Fri, Apr 02, 2010 at 12:48:21PM -0700, Pierre-Loup A. Griffais wrote:
> Signed-off-by: Pierre-Loup A. Griffais <pgriffais at nvidia.com>
> ---
>  hw/xfree86/ramdac/xf86Cursor.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/hw/xfree86/ramdac/xf86Cursor.c b/hw/xfree86/ramdac/xf86Cursor.c
> index 7f23d9e..f5f0873 100644
> --- a/hw/xfree86/ramdac/xf86Cursor.c
> +++ b/hw/xfree86/ramdac/xf86Cursor.c
> @@ -312,6 +312,7 @@ xf86CursorSetCursor(DeviceIntPtr pDev, ScreenPtr pScreen, CursorPtr pCurs,
>              xf86SetCursor(pScreen, NullCursor, x, y);
>              ScreenPriv->isUp = FALSE;
>          }
> +        ScreenPriv->CurrentCursor = NullCursor;
>          return;
>      }
>  

Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

merged into my tree, thanks.

Cheers,
  Peter


More information about the xorg-devel mailing list