On 01.07.2015 04:58 PM, Adam Jackson wrote:
> On failure, AddResource calls the deleteFunc for the given type and
> value.  Here, that's RRModeDestroyResource, which already does
> free(mode) as the last step.  So this patch would introduce a double
> -free.

It does indeed. But only because the ref count is hardcoded to 1...

I haven't checked what AddResource does, probably a big mistake.

Feel free to mark the patch as rejected.



Mihai

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to