tests/bench-slope: fix memory-leak and use-after-free bugs
authorJussi Kivilinna <jussi.kivilinna@iki.fi>
Sat, 21 Mar 2015 11:01:38 +0000 (13:01 +0200)
committerJussi Kivilinna <jussi.kivilinna@iki.fi>
Sat, 21 Mar 2015 11:01:38 +0000 (13:01 +0200)
* tests/bench-slope.c (do_slope_benchmark): Free 'measurements' at end.
(bench_mac_init): Move 'key' free at end of function.
--

Signed-off-by: Jussi Kivilinna <jussi.kivilinna@iki.fi>
tests/bench-slope.c

index c309b7e..394d7fc 100644 (file)
@@ -442,6 +442,7 @@ do_slope_benchmark (struct bench_obj *obj)
               &overhead);
 
   free (measurement_raw);
+  free (measurements);
   free (real_buffer);
   obj->ops->finalize (obj);
 
@@ -1450,11 +1451,11 @@ bench_mac_init (struct bench_obj *obj)
     }
 
   err = gcry_mac_setkey (hd, key, keylen);
-  free (key);
   if (err)
     {
       fprintf (stderr, PGM ": error setting key for mac `%s'\n",
               gcry_mac_algo_name (mode->algo));
+      free (key);
       exit (1);
     }
 
@@ -1473,6 +1474,7 @@ bench_mac_init (struct bench_obj *obj)
 
   obj->priv = hd;
 
+  free (key);
   return 0;
 }