From: David Goulet Date: Thu, 1 Jun 2023 12:35:08 +0000 (-0400) Subject: test: Fix a mem leak reported by Coverity X-Git-Tag: tor-0.4.8.1-alpha~6 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=faff592c3bc4c3bdebd8ac3c24bd23c50dc4c833;p=thirdparty%2Ftor.git test: Fix a mem leak reported by Coverity Here is the report: *** CID 1531835: Resource leaks (RESOURCE_LEAK) /src/test/test_crypto_slow.c: 683 in test_crypto_equix() 677 678 /* Solve phase: Make sure the test vector matches */ 679 memset(&output, 0xa5, sizeof output); 680 equix_result result; 681 result = equix_solve(solve_ctx, challenge_literal, 682 challenge_len, &output); >>> CID 1531835: Resource leaks (RESOURCE_LEAK) >>> Variable "solve_ctx" going out of scope leaks the storage it points to. Signed-off-by: David Goulet --- diff --git a/src/test/test_crypto_slow.c b/src/test/test_crypto_slow.c index 23bc7a852f..3c44b8ded8 100644 --- a/src/test/test_crypto_slow.c +++ b/src/test/test_crypto_slow.c @@ -680,6 +680,7 @@ test_crypto_equix(void *arg) equix_result result; result = equix_solve(solve_ctx, challenge_literal, challenge_len, &output); + equix_free(solve_ctx); tt_int_op(result, OP_EQ, EQUIX_OK); tt_int_op(output.count, OP_EQ, num_sols); tt_int_op(output.flags, OP_EQ, sol_flags);