Re: Clang Static Analyzer reports & more fixes for the Hurd

2013-11-16 Thread Marin Ramesa
On 16.11.2013 10:58:13, Justus Winter wrote: > I've seen Marin has joined in fixing bugs in Mach based on these kind > of analysis, very cool! Yeah, I've been following your reports. It seems this kind of analysis does emit some false alarms, as seen with my recent patch on null pointer derefer

Re: Clang Static Analyzer reports & more fixes for the Hurd

2013-11-16 Thread Samuel Thibault
Justus Winter, le Sat 16 Nov 2013 10:58:13 +0100, a écrit : > Please pay close attention to this one. The Hurd seems to work fine > with this patch. > > [PATCH 6/6] mach-defpager: do not remove -Wall from CFLAGS Well, -Wall shouldn't have any impact on the generated code. Does that really produc

Re: Clang Static Analyzer reports & more fixes for the Hurd

2013-11-16 Thread Samuel Thibault
Justus Winter, le Sat 16 Nov 2013 10:58:13 +0100, a écrit : > The code in libftpconn and libstore/do-bunzip2.c is very noisy. It's probably better to make libstore just use libbz2 than fixing unmaintained code :) Samuel

Clang Static Analyzer reports & more fixes for the Hurd

2013-11-16 Thread Justus Winter
Hi folks :) I've been using the Clang Static Analyzer to analyze the Hurd and GNU Mach source. The results are here (until a more permanent home is found and an automated build system is in place): http://darnassus.sceen.net/~teythoon/qa/ I've seen Marin has joined in fixing bugs in Mach based o