mailing list archives
Re: Clang build with ASAN
From: Alexis La Goutte <alexis.lagoutte () gmail com>
Date: Tue, 13 Aug 2013 21:19:38 +0200
On Tue, Aug 13, 2013 at 8:44 PM, Bálint Réczey <balint () balintreczey hu>wrote:
2013/8/13 Evan Huus <eapache () gmail com>:
On Tue, Aug 13, 2013 at 10:07 AM, Alexis La Goutte
<alexis.lagoutte () gmail com> wrote:
On Tue, Aug 13, 2013 at 3:44 PM, Evan Huus <eapache () gmail com> wrote:
On Tue, Aug 13, 2013 at 6:29 AM, Bálint Réczey <balint () balintreczey hu
2013/8/12 Alexis La Goutte <alexis.lagoutte () gmail com>:
it is now possible to build wireshark with clang (CC=clang
make) (i fix last issue last week end).
I will try the ASAN feature (
ASAN is also ported to GCC 4.8.
I'm not sure which compiler is more reliable regarding ASAN checks,
Also may be now add a clang build to buildbot? (no only scan-build)
I also wanted to raise the question of adding '-fsanitize=address' to
the C flags.
I think fuzzing with ASAN would worth the extra CPU time on buildbots
and I'm even considering
turning ASAN on in the Debian packages by default.
ASAN is basically a borrowed subset of valgrind, so fuzzing with ASAN
not necessary if we can ever get fuzzing with valgrind to work...
ASAN is only 2x slower (valgrind is 20x slower...) and some bugs is only
detect by ASAN...
I knew it would be faster, but I didn't know it also detected stack
(and not just heap errors). However, it does not detect uninitialized
(which Valgrind does) so for full coverage we need to do both.
Yes, Valgrind still have better coverage and also capable of doing
other nice things.
Maybe fuzzing with ASAN often (like daily) and fuzzing with Valgrind
(weekly) would be
a good compromise resulting catching most problems quickly and the
rest with a few days of lag.
My first idea is add a (real) clang buildbot (Only to check if wireshark
always build with clang :-))
Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org>
mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Re: Clang build with ASAN Evan Huus (Aug 13)