[PATCH] render: Fix double-free on ARGB cursor error path

Keith Packard keithp at keithp.com
Thu Sep 11 17:49:49 PDT 2014


Adam Jackson <ajax at redhat.com> writes:

> The gotos deleted by this patch are the only way to get to the bail:
> label here.  In neither case do we need to free the cursor bits from the
> caller; AllocARGBCursor will already do that on the failure path,
> likewise AddResource will call the resource delete function on error.

This patch points out an obvious bug, but the fix isn't correct yet (as
commented on already). Do you want to hack on it some more so we can get
it merged?

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20140911/94ff3894/attachment.sig>


More information about the xorg-devel mailing list