<html><head></head><body>Hi all,<div><br></div><div>I believe we are affected by this bug and had to downgrade to 2.89 on our Debian installation.</div><div><br></div><div>I see it has been a few months since the last message regarding this issue. I'm just a normal user and a little bit confused on how to proceed.</div><div>Is this something that is still being worked on, and is there an expectation of when the new version will be released that fixes this issue? Or is there something wrong with the program performing the DNS requests and should I ask them to fix something on their side to handle what is happening here?</div><div><br></div><div>Thanks in advance!</div><div>Niels Hendriks<br><br><br><div><strong>
From:
</strong>
 
Rahul Thakur via Dnsmasq-discuss <dnsmasq-discuss@lists.thekelleys.org.uk>
<br>
<strong>
To:
</strong>
 
<dnsmasq-discuss@lists.thekelleys.org.uk>
<br>
<strong>
Sent:
 
</strong>
04/10/2024 1:33 PM
<br>
<strong>
Subject:
</strong>
 
[Dnsmasq-discuss] [PATCH] forward.c: fix handling of truncated response
<br><br><blockquote class="mori" style="margin:0 0 0 .8ex;border-left:1px solid #CCC;padding-left:1ex;">From: Rahul Thakur <rahul.thakur@iopsys.eu>
<br>
<br>The handling of truncated reponse is broken in 2.90. The answers
<br>are removed before forwarding in case TC bit is set, which
<br>seems incorrect as per rfc 5625.
<br>A combined reading of section 4.4.1 of rfc 5625 section 6.1.3.2
<br>of rfc 1123 suggests when dnsmasq explicitly deletes the answers
<br>in a truncated response, then it deprives clients that do not fallback
<br>to TCP and are happy with the truncated UDP response to be able
<br>to resolve the queries. In light of this, it sounds like a better
<br>strategy is to forward the truncated UDP response as is to the client,
<br>and let the client decide what action it wants to take.
<br>
<br>Signed-off-by: Rahul Thakur <rahul.thakur@iopsys.eu>
<br>---
<br> src/forward.c | 3 ---
<br> 1 file changed, 3 deletions(-)
<br>
<br>diff --git a/src/forward.c b/src/forward.c
<br>index 10e7496..1ede913 100644
<br>--- a/src/forward.c
<br>+++ b/src/forward.c
<br>@@ -785,9 +785,6 @@ static size_t process_reply(struct dns_header *header, time_t now, struct server
<br>   if (header->hb3 & HB3_TC)
<br>     {
<br>       log_query(F_UPSTREAM, NULL, NULL, "truncated", 0);
<br>-      header->ancount = htons(0);
<br>-      header->nscount = htons(0);
<br>-      header->arcount = htons(0);
<br>     }
<br> 
<br>   if  (!(header->hb3 & HB3_TC) && (!bogusanswer || (header->hb4 & HB4_CD)))
<br>-- 
<br>2.25.1
<br>
<br>
<br>_______________________________________________
<br>Dnsmasq-discuss mailing list
<br>Dnsmasq-discuss@lists.thekelleys.org.uk
<br>https://lists.thekelleys.org.uk/cgi-bin/mailman/listinfo/dnsmasq-discuss
<br></blockquote></div></div></body></html>