[PATCH]: fix refcounting in xf86_use_hw_cursor

Roland Scheidegger rscheidegger_lists at hispeed.ch
Wed Mar 10 09:43:00 PST 2010


On 10.03.2010 12:01, Michel Dänzer wrote:
> 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)
sorry my split (email) personality problem...

Ok I've resent everything hopefully correct this time...


Roland

> 
> 
>> 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;
> 
> 
> 
> 



More information about the xorg-devel mailing list