commit f5d5fba52c9d0bc2f08b78415633e11e06a94e1c
Author: David Goulet <[email protected]>
Date: Wed Oct 1 16:24:33 2014 -0400
Test: possible double free in onion test
Fix Coverity issue 1195191.
Signed-off-by: David Goulet <[email protected]>
---
tests/unit/test_onion.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/tests/unit/test_onion.c b/tests/unit/test_onion.c
index c29d5e9..98bcd63 100644
--- a/tests/unit/test_onion.c
+++ b/tests/unit/test_onion.c
@@ -114,6 +114,7 @@ static void test_onion_init(struct onion_pool *pool)
pool->count == 0 &&
pool->next_entry_pos == 0,
"Valid onion pool of %s/%d", DEFAULT_ONION_ADDR_RANGE, mask);
+ onion_pool_destroy(pool);
/* Valid test. */
base = inet_addr("127.42.42.64");
@@ -127,6 +128,7 @@ static void test_onion_init(struct onion_pool *pool)
pool->count == 0 &&
pool->next_entry_pos == 0,
"Valid onion pool of 127.42.42.64/27");
+ onion_pool_destroy(pool);
/* Valid test. */
base = inet_addr("127.42.42.64");
@@ -140,6 +142,7 @@ static void test_onion_init(struct onion_pool *pool)
pool->count == 0 &&
pool->next_entry_pos == 0,
"Valid onion pool of 127.42.42.64/17");
+ onion_pool_destroy(pool);
/* Valid test with size less than default. */
base = inet_addr("127.42.42.0");
@@ -153,6 +156,7 @@ static void test_onion_init(struct onion_pool *pool)
pool->count == 0 &&
pool->next_entry_pos == 0,
"Valid onion pool of 127.42.42.0/32");
+ onion_pool_destroy(pool);
/* Invalid test. */
base = inet_addr("127.42.42.64");
@@ -160,8 +164,6 @@ static void test_onion_init(struct onion_pool *pool)
ret = onion_pool_init(pool, base, mask);
ok(ret == -EINVAL,
"Invalid onion pool of mask 42");
-
- onion_pool_destroy(pool);
}
int main(int argc, char **argv)
_______________________________________________
tor-commits mailing list
[email protected]
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
