From: Sven Haardiek <[email protected]> Some DNS Queries include an optional RR in the additional record section of DNS, so simply copying the non-zero DNS Header for the additional records, but not adding any leads to broken DNS packages.
This patch explicitly sets the additional records entry in the DNS header to 0. Closes #114 Submitted-at: https://github.com/ovn-org/ovn/pull/115 Signed-off-by: Sven Haardiek <[email protected]> --- controller/pinctrl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/controller/pinctrl.c b/controller/pinctrl.c index 293aecea2..fd0bccdb6 100644 --- a/controller/pinctrl.c +++ b/controller/pinctrl.c @@ -3023,8 +3023,9 @@ pinctrl_handle_dns_lookup( /* Set the response bit to 1 in the flags. */ out_dns_header->lo_flag |= 0x80; - /* Set the answer RR. */ + /* Set the answer RRs. */ out_dns_header->ancount = htons(ancount); + out_dns_header->arcount = 0; /* Copy the Query section. */ dp_packet_put(&pkt_out, dp_packet_data(pkt_in), dp_packet_size(pkt_in)); -- 2.34.1 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
