nihilist/tor
1
0
Fork 0
mirror of https://gitlab.torproject.org/tpo/core/tor.git synced 2024-11-10 05:03:43 +01:00
Code Issues Packages Projects Releases Wiki Activity

test: Fix a mem leak reported by Coverity

Browse Source 
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>
This commit is contained in:
David Goulet 2023-06-01 08:35:08 -04:00
parent 97008526db
commit faff592c3b
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/test/test_crypto_slow.c
View File

@ -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);