diff options
| author | David Goulet <dgoulet@torproject.org> | 2023-06-01 08:35:08 -0400 |
|---|---|---|
| committer | David Goulet <dgoulet@torproject.org> | 2023-06-01 08:35:08 -0400 |
| commit | faff592c3bc4c3bdebd8ac3c24bd23c50dc4c833 (patch) | |
| tree | ce616b85a3bcdfd337dbce20000b2fa66cb9afcf | |
| parent | 97008526db53ce77cc65fbf93a4f5760c242082b (diff) | |
| download | tor-faff592c3bc4c3bdebd8ac3c24bd23c50dc4c833.tar.gz tor-faff592c3bc4c3bdebd8ac3c24bd23c50dc4c833.zip | |
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 <dgoulet@torproject.org>
| -rw-r--r-- | src/test/test_crypto_slow.c | 1 |
1 files changed, 1 insertions, 0 deletions
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); |