[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:
http://people.freedesktop.org/~jeremyhu/analyzer/tifa-linux32/20110927-1930

Some drivers didn't build though due to the swap macro change, but the rest look to have good results:
http://tinderbox.x.org/builds/2011-09-28-0004

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