diff options
author | Jussi Kivilinna <jussi.kivilinna@iki.fi> | 2015-03-21 13:01:38 +0200 |
---|---|---|
committer | Jussi Kivilinna <jussi.kivilinna@iki.fi> | 2015-03-21 13:01:38 +0200 |
commit | aa234561d00c3fb15fe501df4bf58f3db7c7c06b (patch) | |
tree | f464361bff7182eea5c453731de76187cc691c1d | |
parent | f5832285b0e420d77be1b8da10a1e1d86583b414 (diff) | |
download | libgcrypt-aa234561d00c3fb15fe501df4bf58f3db7c7c06b.tar.gz |
tests/bench-slope: fix memory-leak and use-after-free bugs
* 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>
-rw-r--r-- | tests/bench-slope.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tests/bench-slope.c b/tests/bench-slope.c index c309b7e6..394d7fc0 100644 --- a/tests/bench-slope.c +++ b/tests/bench-slope.c @@ -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; } |