Re: [PATCH 01/12] dix: fix double free around allocargbcursor.

2011-10-20 Thread Alan Coopersmith
On 10/20/11 03:44, Dave Airlie wrote: From: Dave Airlie coverity scan pointed this out. In some of its error cases AllocARGBCursor freed the bits passed in, because it assigned them to the cursor and called FreeCursorBits, in one case it didn't, if it hadn't done the assignment. This standardi

[PATCH 01/12] dix: fix double free around allocargbcursor.

2011-10-20 Thread Dave Airlie
From: Dave Airlie coverity scan pointed this out. In some of its error cases AllocARGBCursor freed the bits passed in, because it assigned them to the cursor and called FreeCursorBits, in one case it didn't, if it hadn't done the assignment. This standardises the interface so it always frees in