diff options
author | Jeremy Harris <jgh146exb@wizmail.org> | 2021-05-05 20:14:47 +0100 |
---|---|---|
committer | Jeremy Harris <jgh146exb@wizmail.org> | 2021-05-05 22:04:13 +0100 |
commit | 143b1159402424b930881bc8d25c640f1d6cd581 (patch) | |
tree | 899b1fbe1eace77a8a13f678d0661f1138801947 /src | |
parent | 8e61d2807317ed66163a395c1cd8666fd442d9aa (diff) |
wip
Diffstat (limited to 'src')
-rw-r--r-- | src/src/dns.c | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/src/src/dns.c b/src/src/dns.c index c68698786..217fe5f87 100644 --- a/src/src/dns.c +++ b/src/src/dns.c @@ -691,9 +691,11 @@ packet length has been lost inside libresolv, so we have to guess a replacement value. (The only way to fix this properly would be to re-implement res_search() and res_query() so that they don't muddle their success and packet length return values.) For added safety we only reset -the packet length if the packet header looks plausible. */ +the packet length if the packet header looks plausible. -static void +Return TRUE iff it seemed ok */ + +static BOOL fake_dnsa_len_for_fail(dns_answer * dnsa, int type) { const HEADER * h = (const HEADER *)dnsa->answer; @@ -710,7 +712,11 @@ if ( h->qr == 1 /* a response */ DEBUG(D_dns) debug_printf("faking res_search(%s) response length as %d\n", dns_text_type(type), (int)sizeof(dnsa->answer)); dnsa->answerlen = sizeof(dnsa->answer); + return TRUE; } +DEBUG(D_dns) debug_printf("DNS: couldn't fake dnsa len\n"); +/* Maybe we should just do a second lookup for an SOA? */ +return FALSE; } @@ -724,7 +730,7 @@ dns_expire_from_soa(dns_answer * dnsa, int type) { dns_scan dnss; -fake_dnsa_len_for_fail(dnsa, type); +if (!fake_dnsa_len_for_fail(dnsa, type)) return 0; for (dns_record * rr = dns_next_rr(dnsa, &dnss, RESET_AUTHORITY); rr; rr = dns_next_rr(dnsa, &dnss, RESET_NEXT) @@ -1201,7 +1207,7 @@ switch (type) if (rc == DNS_NOMATCH) { - fake_dnsa_len_for_fail(dnsa, T_CSA); + if (!fake_dnsa_len_for_fail(dnsa, T_CSA)) return DNS_NOMATCH; for (rr = dns_next_rr(dnsa, &dnss, RESET_AUTHORITY); rr; rr = dns_next_rr(dnsa, &dnss, RESET_NEXT) |