clang static analysis of xorg-server

Alan Coopersmith Alan.Coopersmith at Sun.COM
Mon Mar 22 14:08:15 PDT 2010


Jeremy Huddleston wrote:
> I aree ... some of the "dead store" issues are more stylistic or future proofing.
> 
> The real ones we should consider are the logic errors... garbage assignment and null dereference.
> 
> Some of the errors seem outright bogus (http://people.freedesktop.org/~jeremyhu/clang/2010-03-22-1/report-K6CxIC.html#EndPath), so if you see any others like that, let me know.  I'm building the latest version of clang right now, and if those oddities are still present, I'll report them to the developers.

That might be pointing out that the definition of XkbControlsEnabledMask is
technically undefined on a 32-bit system, since it should be (1UL << 31) not
(1L << 31) - we've gotten that error in various tools before.

-- 
	-Alan Coopersmith-           alan.coopersmith at sun.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list