]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Change the default EDNS buffer size for dig, mdig and host to 1232
authorOndřej Surý <ondrej@isc.org>
Wed, 30 Sep 2020 16:18:05 +0000 (18:18 +0200)
committerOndřej Surý <ondrej@sury.org>
Mon, 5 Oct 2020 14:21:21 +0000 (16:21 +0200)
The dig should behave closely to how the resolver code in `named` work,
so we need to change the buffer size in dig, mdig and host too.

bin/dig/dighost.h
bin/tests/system/digdelv/tests.sh
bin/tools/mdig.c

index 385b9abcf102f32730da364432092b630ed6f9a4..bc280e540e9a74cf6f2c2647844eb9cef6a5e418 100644 (file)
@@ -73,7 +73,7 @@
 #define LOOKUP_LIMIT 64
 
 #define DEFAULT_EDNS_VERSION 0
-#define DEFAULT_EDNS_BUFSIZE 4096
+#define DEFAULT_EDNS_BUFSIZE 1232
 
 /*%
  * Lookup_limit is just a limiter, keeping too many lookups from being
index 99b80992f2a4874a0dacb066d3c506298432ee28..aed4c23ede9fdee6c1f5f91c4da85a6408880a7c 100644 (file)
@@ -958,10 +958,10 @@ if [ -x "$DIG" ] ; then
   echo_i "check that dig +bufsize restores default bufsize ($n)"
   ret=0
   dig_with_opts @10.53.0.3 a.example +bufsize=0 +bufsize +qr > dig.out.test$n 2>&1 || ret=1
+  lines=`grep "EDNS:.* udp:" dig.out.test$n | wc -l`
   lines1232=`grep "EDNS:.* udp: 1232" dig.out.test$n | wc -l`
-  lines4096=`grep "EDNS:.* udp: 4096" dig.out.test$n | wc -l`
-  test $lines1232 -eq 1 || ret=1
-  test $lines4096 -eq 1 || ret=1
+  test $lines -eq 2 || ret=1
+  test $lines1232 -eq 2 || ret=1
   if [ $ret -ne 0 ]; then echo_i "failed"; fi
   status=$((status+ret))
 
index c6e4ea43160b0c9dfa1dc7400d3d65ca537e113c..a69b1e4e8ce610308b9f7ab3fc5848649030f97a 100644 (file)
@@ -638,7 +638,7 @@ sendquery(struct query *query, isc_task_t *task) {
                unsigned char cookie[40];
 
                if (query->udpsize == 0) {
-                       query->udpsize = 4096;
+                       query->udpsize = 1232;
                }
                if (query->edns < 0) {
                        query->edns = 0;