Double-free in crypto tests on failure

Review Request #1127 - Created July 7, 2018 and submitted

Jason King

If a crypto test fails, it will try to free some of the memory it allocated twice. This is easy to trigger by interposing libumem while running the crypto tests.

Running with libumem interposed + UMEM_DEBUG=default on the crypto tests will cause the AES CCM and GCM tests to crash due to the double free, when the should just exit with failure (due to not currently implementing them). With the fix, they fail due to lack of support as expected and do not crash when libumem is interposed.

Toomas Soome
Andy Fiddaman
Yuri Pankov
Andrew Stormont
Gergő Mihály Doma
Jason King
Review request changed

Status: Closed (submitted)