From 18b6d475dc160efba24861e584dd6de92a573862 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Tue, 14 Jun 2011 16:46:13 +0200 Subject: [PATCH] Do not leak rr_name on failures inside ares_parse_aaaa_reply --- ares_parse_aaaa_reply.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/ares_parse_aaaa_reply.c b/ares_parse_aaaa_reply.c index a0f524daa..1fbe8389f 100644 --- a/ares_parse_aaaa_reply.c +++ b/ares_parse_aaaa_reply.c @@ -132,6 +132,7 @@ int ares_parse_aaaa_reply(const unsigned char *abuf, int alen, aptr += len; if (aptr + RRFIXEDSZ > abuf + alen) { + free(rr_name); status = ARES_EBADRESP; break; } @@ -149,6 +150,7 @@ int ares_parse_aaaa_reply(const unsigned char *abuf, int alen, { if (aptr + sizeof(struct ares_in6_addr) > abuf + alen) { + free(rr_name); status = ARES_EBADRESP; break; } @@ -159,6 +161,7 @@ int ares_parse_aaaa_reply(const unsigned char *abuf, int alen, struct ares_addr6ttl * const at = &addrttls[naddrs]; if (aptr + sizeof(struct ares_in6_addr) > abuf + alen) { + free(rr_name); status = ARES_EBADRESP; break; }