summaryrefslogtreecommitdiff
path: root/libc
diff options
context:
space:
mode:
authorBernhard Reutner-Fischer <rep.dot.nop@gmail.com>2011-10-14 17:47:42 +0200
committerBernhard Reutner-Fischer <rep.dot.nop@gmail.com>2011-10-21 08:23:22 +0200
commitbb8d500a75a3050fe3198773ce7b07f669fe8f13 (patch)
tree9b0ced7eb4ae8bbe2e6ea8ccc9d8f99922568a0a /libc
parente3fea65f50c6e5e82e75b8940dcf857495136e9a (diff)
resolv: fix memory leak
Timothy Holdener writes: small memory leak in __dns_lookup() when the A record in the DNS answer is preceded by one or more CNAME records. Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
Diffstat (limited to 'libc')
-rw-r--r--libc/inet/resolv.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/libc/inet/resolv.c b/libc/inet/resolv.c
index 38254bdfd..e415d30a5 100644
--- a/libc/inet/resolv.c
+++ b/libc/inet/resolv.c
@@ -1507,6 +1507,7 @@ int attribute_hidden __dns_lookup(const char *name,
DPRINTF("Decoding answer at pos %d\n", pos);
first_answer = 1;
+ a->dotted = NULL;
for (j = 0; j < h.ancount; j++) {
i = __decode_answer(packet, pos, packet_len, &ma);
if (i < 0) {
@@ -1523,6 +1524,7 @@ int attribute_hidden __dns_lookup(const char *name,
ma.buf = a->buf;
ma.buflen = a->buflen;
ma.add_count = a->add_count;
+ free(a->dotted);
memcpy(a, &ma, sizeof(ma));
if (a->atype != T_SIG && (NULL == a->buf || (type != T_A && type != T_AAAA)))
break;