Adam Jackson <[email protected]> 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.
I don't see AllocARGBCursor freeing anything in the failure paths, so I think we need to free source/mask. We also need to free argbits However, AddResource does clean things up nicely so there's no need to free things from that path. -- [email protected]
pgp_YVeUYoMwY.pgp
Description: PGP signature
_______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
