[ovs-dev] [PATCH 10/10] tests: Add support for Address Sanitizer.

[Ben Pfaff]

On Sat, Dec 01, 2018 at 11:56:55AM -0800, Justin Pettit wrote:
> 
> > On Nov 14, 2018, at 10:05 PM, Ben Pfaff <blp at ovn.org> wrote:
> > 
> > diff --git a/tests/atlocal.in b/tests/atlocal.in
> > index abfc1acf36d3..6f971cd652d7 100644
> > --- a/tests/atlocal.in
> > +++ b/tests/atlocal.in
> > @@ -220,3 +220,8 @@ export OVS_SYSLOG_METHOD
> > # Set default timeout for control utils
> > OVS_CTL_TIMEOUT=30
> > export OVS_CTL_TIMEOUT
> > +
> > +# Add some default flags to make the tests run better under Address
> > +# Sanitizer, if it was used for the build.
> > +ASAN_OPTIONS=detect_leaks=0:abort_on_error=true:log_path=asan:$ASAN_OPTIONS
> 
> Is the leak detection disabled due to false positives?  It may be worth mentioning the reason.

Good idea, I added a note.

> Thanks for adding this.
> 
> Acked-by: Justin Pettit <jpettit at ovn.org>

Thanks, applied to master.