tree 8ec677c48523a26f30b84e7ae6dbf5e4dd6b6370
parent 093aa517c47d7388866f1fddd251184e9d829d08
author Gilles Peskine <Gilles.Peskine@arm.com> 1572280126 +0100
committer Gilles Peskine <Gilles.Peskine@arm.com> 1572293228 +0100

Fix CTR_DRBG benchmark

You can't reuse a CTR_DRBG context without free()ing it and
re-init()ing it. This generally happened to work, but was never
guaranteed. It could have failed with alternative implementations of
the AES module because mbedtls_ctr_drbg_seed() calls
mbedtls_aes_init() on a context which is already initialized if
mbedtls_ctr_drbg_seed() hasn't been called before, plausibly causing a
memory leak. Calling free() and seed() with no intervening init fails
when MBEDTLS_THREADING_C is enabled and all-bits-zero is not a valid
mutex representation. So add the missing free() and init().