[PATCH]: fix refcounting in xf86_use_hw_cursor

Michel Dänzer michel at daenzer.net
Wed Mar 10 03:01:19 PST 2010


On Tue, 2010-03-09 at 18:25 +0100, Roland Scheidegger wrote: 
> This is the same fix as was done in
> fcdc1d78cca3b8bb6b77d53eda7e21d649df6943 for xf86_use_hw_cursor_argb.

Reviewed-by: Michel Dänzer <michel at daenzer.net>

Roland, you need to add Signed-off-by: tags to your patches, or Keith
may just ignore them. (Also consider using git send-email, e.g. I
suspect your author e-mail address should be different)


> diff --git a/hw/xfree86/modes/xf86Cursors.c b/hw/xfree86/modes/xf86Cursors.c
> index 385848b..e2e174e 100644
> --- a/hw/xfree86/modes/xf86Cursors.c
> +++ b/hw/xfree86/modes/xf86Cursors.c
> @@ -461,11 +461,11 @@ xf86_use_hw_cursor (ScreenPtr screen, CursorPtr cursor)
>      xf86CrtcConfigPtr   xf86_config = XF86_CRTC_CONFIG_PTR(scrn);
>      xf86CursorInfoPtr	cursor_info = xf86_config->cursor_info;
>  
> +    ++cursor->refcnt;
>      if (xf86_config->cursor)
>  	FreeCursor (xf86_config->cursor, None);
>      xf86_config->cursor = cursor;
> -    ++cursor->refcnt;
> -    
> +
>      if (cursor->bits->width > cursor_info->MaxWidth ||
>  	cursor->bits->height> cursor_info->MaxHeight)
>  	return FALSE;




-- 
Earthling Michel Dänzer           |                http://www.vmware.com
Libre software enthusiast         |          Debian, X and DRI developer


More information about the xorg-devel mailing list