No longer uses the valgrind option '--logfile-fd', we use the --logfile

option instead (even though it appends the pid to the file name, making it
harder to figure out its name to parse it after a test has run).

Also made sure we only use valgrind for the actual test command command lines,
not when for example running curl to detect if there are any already running
servers are present etc.

diff --git a/tests/runtests.pl b/tests/runtests.pl
index 8de3c06..2d9a97e 100755 (executable)
--- a/tests/runtests.pl
+++ b/tests/runtests.pl
@@ -1024,7 +1024,7 @@ sub singletest {
     }
 
     if($valgrind) {
-        $CMDLINE = "exec 3>log/valgrind$testnum && $CMDLINE";
+        $CMDLINE = "valgrind --leak-check=yes --logfile=log/valgrind$testnum -q $CMDLINE";
     }
 
     $CMDLINE .= "$cmdargs >>$STDOUT 2>>$STDERR";
@@ -1476,9 +1476,6 @@ if($valgrind) {
         #print "Valgrind failure, disable it\n";
         undef $valgrind;
     }
-    else {
-        $CURL="valgrind --leak-check=yes --logfile-fd=3 -q $CURL";
-    }
 }
 
 #######################################################################