On Wed, Nov 25, 2020 at 8:31 PM Dumitru Ceara <[email protected]> wrote: > > Recent commits cleaned up most of the minor leaks that would've made > tests with AddressSanitizer memory leak checking enabled fail. It > should be safe (and useful) to check for memory leaks by default. > > Signed-off-by: Dumitru Ceara <[email protected]>
Thanks Dumitru. This will be very useful. I applied this patch to master. Once we have github actions patch merged, we can make use of this. Thanks Numan > --- > tests/atlocal.in | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/tests/atlocal.in b/tests/atlocal.in > index 4517ebf..d9a4c91 100644 > --- a/tests/atlocal.in > +++ b/tests/atlocal.in > @@ -205,8 +205,5 @@ export OVS_CTL_TIMEOUT > > # Add some default flags to make the tests run better under Address > # Sanitizer, if it was used for the build. > -# > -# We disable leak detection because otherwise minor leaks that don't > -# matter break everything. > -ASAN_OPTIONS=detect_leaks=0:abort_on_error=true:log_path=asan:$ASAN_OPTIONS > +ASAN_OPTIONS=detect_leaks=1:abort_on_error=true:log_path=asan:$ASAN_OPTIONS > export ASAN_OPTIONS > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
