From: Nick M. <ni...@us...> - 2010-03-13 06:28:27
|
Author: Nick Mathewson <ni...@to...> Date: Sat, 13 Mar 2010 00:53:54 -0500 Subject: Free evdns_base->req_heads on evdns_base_free Commit: 859af6772c8ff929ec73b862bb3b64dc0e57bada It looks like when we moved from one big inflight-requests list to an n-heads structure, we didn't make evdns_base_free() free the array of heads. This patch should fix that. Found with valgrind --- evdns.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/evdns.c b/evdns.c index e9cbec7..88d6fa3 100644 --- a/evdns.c +++ b/evdns.c @@ -3775,6 +3775,8 @@ evdns_base_free_and_unlock(struct evdns_base *base, int fail_requests) } } + mm_free(base->req_heads); + EVDNS_UNLOCK(base); EVTHREAD_FREE_LOCK(base->lock, EVTHREAD_LOCKTYPE_RECURSIVE); -- 1.6.3 |