[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