[PATCH xserver 1/2] animcur: Fix transitions between animated cursors

Adam Jackson ajax at redhat.com
Tue Jan 9 22:16:23 UTC 2018


On Tue, 2018-01-09 at 14:06 -0800, Keith Packard wrote:
> Adam Jackson <ajax at redhat.com> writes:
> 
> > +static void
> > +AnimCurCancelTimer(DeviceIntPtr pDev)
> > +{
> > +    CursorPtr cur = pDev->spriteInfo->anim.pCursor;
> > +
> > +    if (IsAnimCur(cur))
> > +        TimerCancel(GetAnimCur(cur)->timer);
> > +}
> > +
> 
> Should check for a NULL cursor here.

It does, hidden in the macro:

render/animcur.c:#define IsAnimCur(c)       ((c) && ((c)->bits == &animCursorBits))

- ajax


More information about the xorg-devel mailing list