From 186a5195b54cb9cae77fbfecc3cd478b8fb8a68b Mon Sep 17 00:00:00 2001 From: James Prestwood Date: Wed, 13 Nov 2019 11:41:02 -0800 Subject: [PATCH] test-runner: erase valgrind log after each test After changing the valgrind log to --log-fd, it appears that the log file just gets appended, where before it was overwritten. This makes test print out all previous tests valgrind results. Now after printing out the valgrind info we can erase the file for the next test. --- tools/test-runner.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tools/test-runner.c b/tools/test-runner.c index 043e8cc7..075f56d6 100644 --- a/tools/test-runner.c +++ b/tools/test-runner.c @@ -2226,10 +2226,14 @@ static void create_network_and_run_tests(void *data, void *user_data) terminate_iwd(iwd_pid); /* /tmp/valgrind.log will only exist without logging turned on */ - if (valgrind && !log) + if (valgrind && !log) { if (system("cat /tmp/valgrind.log")) l_info("cat /tmp/valgrind.log failed"); + if (system("echo \"\" > /tmp/valgrind.log")) + l_info("Failed to reset /tmp/valgrind.log"); + } + if (log) { L_AUTO_FREE_VAR(char *, dmesg); L_AUTO_FREE_VAR(char *, kernel_log);