commit f5d5fba52c9d0bc2f08b78415633e11e06a94e1c Author: David Goulet dgoulet@ev0ke.net 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 dgoulet@ev0ke.net --- 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)