From: Christian Hopps <chopps@labn.net>
Date: Sat, 22 Apr 2023 16:59:53 +0000 (-0400)
Subject: tests: fix ASAN file cleanup and --asan-abort option
X-Git-Tag: base_9.0~147^2~2
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=449e25552bc9b9a4b1d4c918fc8f8f7e21e5b8b5;p=matthieu%2Ffrr.git

tests: fix ASAN file cleanup and --asan-abort option

Signed-off-by: Christian Hopps <chopps@labn.net>
---

diff --git a/tests/topotests/lib/topotest.py b/tests/topotests/lib/topotest.py
index bfa3e59074..93daf0bd8c 100644
--- a/tests/topotests/lib/topotest.py
+++ b/tests/topotests/lib/topotest.py
@@ -1055,7 +1055,7 @@ def checkAddressSanitizerError(output, router, component, logdir=""):
 
     # No Address Sanitizer Error in Output. Now check for AddressSanitizer daemon file
     if logdir:
-        filepattern = logdir + "/" + router + "/" + component + ".asan.*"
+        filepattern = logdir + "/" + router + ".asan." + component + ".*"
         logger.debug(
             "Log check for %s on %s, pattern %s\n" % (component, router, filepattern)
         )
@@ -1272,7 +1272,8 @@ def fix_host_limits():
 def setup_node_tmpdir(logdir, name):
     # Cleanup old log, valgrind, and core files.
     subprocess.check_call(
-        "rm -rf {0}/{1}.valgrind.* {1}.*.asan {0}/{1}/".format(logdir, name), shell=True
+        "rm -rf {0}/{1}.valgrind.* {0}/{1}.asan.* {0}/{1}/".format(logdir, name),
+        shell=True,
     )
 
     # Setup the per node directory.
@@ -1771,8 +1772,8 @@ class Router(Node):
 
                 cmdenv = "ASAN_OPTIONS="
                 if asan_abort:
-                    cmdenv = "abort_on_error=1:"
-                cmdenv += "log_path={0}/{1}.{2}.asan ".format(
+                    cmdenv += "abort_on_error=1:"
+                cmdenv += "log_path={0}/{1}.asan.{2} ".format(
                     self.logdir, self.name, daemon
                 )