|
|
Message-ID: <ZR29J211K8CTWWbm@fifth.space>
Date: Wed, 4 Oct 2023 21:29:43 +0200
From: Quentin Rameau <quinq@...th.space>
To: musl@...ts.openwall.com
Subject: [PATCH] Remove arbitrary limit from NS resolution
The name resolution would abort when getting more than 63 records per
request, due to what seems to be a left-over from the original code.
Somehow reported by Daniel Stefanik[0].
[0] https://gitlab.alpinelinux.org/alpine/aports/-/issues/15320
---
src/network/dns_parse.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/network/dns_parse.c b/src/network/dns_parse.c
index ea1ec126..09813112 100644
--- a/src/network/dns_parse.c
+++ b/src/network/dns_parse.c
@@ -12,7 +12,6 @@ int __dns_parse(const unsigned char *r, int rlen, int (*callback)(void *, int, c
p = r+12;
qdcount = r[4]*256 + r[5];
ancount = r[6]*256 + r[7];
- if (qdcount+ancount > 64) return -1;
while (qdcount--) {
while (p-r < rlen && *p-1U < 127) p++;
if (p>r+rlen-6)
--
2.42.0
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.