[PATCH] Remove another if (E != NULL) check around free(E)

Jeremy Huddleston jeremyhu at apple.com
Wed Sep 28 08:37:07 PDT 2011

This looks like a pretty good run of the analyzer:

Some drivers didn't build though due to the swap macro change, but the rest look to have good results:

On Sep 27, 2011, at 11:30 PM, Jeremy Huddleston wrote:

> On Sep 26, 2011, at 7:58 PM, Matt Turner wrote:
>> I wonder if there are any other patterns we haven't seen yet?
> Well clang static analyzer should help with that, but I just realized it was having issues.  I'll try to fix it tomorrow for tomorrow evening's tinderbox run.
> --Jeremy
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel

Jeremy Huddleston

Rebuild Sudan
 - Board of Directors
 - http://www.rebuildsudan.org

Berkeley Foundation for Opportunities in Information Technology
 - Advisory Board
 - http://www.bfoit.org

More information about the xorg-devel mailing list