tests: Fix a clang possible memory leak warning

clang was reporting a possible memory leak after mkdtemp() call, which
was a false positive, since mkdtemp() returns the same pointer provided
as the parameter, in case of success.  This changes the code so that the
static analyser don't get confused.

Found by csbuild runner.

Signed-off-by: Anderson Toshiyuki Sasaki <ansasaki@redhat.com>
Reviewed-by: Andreas Schneider <asn@cryptomilk.org>

tests/torture.c

@@ -805,11 +805,11 @@ char *torture_make_temp_dir(const char *template)
 
     new_dir = mkdtemp(template_copy);
     if (new_dir == NULL) {
-        free(template_copy);
+        SAFE_FREE(template_copy);
     }
 
 end:
-    return new_dir;
+    return template_copy;
 }
 
 char *torture_create_temp_file(const char *template)