From: Ondřej Surý <ondrej@isc.org>
Date: Thu, 11 Feb 2021 13:25:58 +0000 (+0100)
Subject: Stop including dnstap headers from <dns/dnstap.h>
X-Git-Tag: v9.17.11~52^2~1
X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=23c3bcc71191ae25d10e45284e7e44f982fcfff4;p=thirdparty%2Fbind9.git

Stop including dnstap headers from <dns/dnstap.h>

The <fstrm.h> and <protobuf-c/protobuf-c.h> headers are only directly
included where used and we stopped exposing those headers from libdns
headers.
---

diff --git a/bin/named/main.c b/bin/named/main.c
index cfdc941e055..c8ea183269e 100644
--- a/bin/named/main.c
+++ b/bin/named/main.c
@@ -18,6 +18,10 @@
 #include <string.h>
 #include <uv.h>
 
+#ifdef HAVE_DNSTAP
+#include <protobuf-c/protobuf-c.h>
+#endif
+
 #include <isc/app.h>
 #include <isc/attributes.h>
 #include <isc/backtrace.h>
diff --git a/bin/named/server.c b/bin/named/server.c
index c2afd5523a0..4659815ffb7 100644
--- a/bin/named/server.c
+++ b/bin/named/server.c
@@ -20,6 +20,10 @@
 #include <sys/types.h>
 #include <unistd.h>
 
+#ifdef HAVE_DNSTAP
+#include <fstrm.h>
+#endif
+
 #include <isc/aes.h>
 #include <isc/app.h>
 #include <isc/attributes.h>
diff --git a/lib/bind9/check.c b/lib/bind9/check.c
index e789aaadd1c..67f2f96642f 100644
--- a/lib/bind9/check.c
+++ b/lib/bind9/check.c
@@ -15,6 +15,10 @@
 #include <stdbool.h>
 #include <stdlib.h>
 
+#ifdef HAVE_DNSTAP
+#include <fstrm.h>
+#endif
+
 #include <isc/aes.h>
 #include <isc/base64.h>
 #include <isc/buffer.h>
diff --git a/lib/dns/dnstap.c b/lib/dns/dnstap.c
index 44fb25e73ce..2688b30a338 100644
--- a/lib/dns/dnstap.c
+++ b/lib/dns/dnstap.c
@@ -47,12 +47,11 @@
 #error DNSTAP not configured.
 #endif /* HAVE_DNSTAP */
 
+#include <fstrm.h>
 #include <inttypes.h>
 #include <stdbool.h>
 #include <stdlib.h>
 
-#include <protobuf-c/protobuf-c.h>
-
 #include <isc/buffer.h>
 #include <isc/file.h>
 #include <isc/log.h>
diff --git a/lib/dns/include/dns/dnstap.h b/lib/dns/include/dns/dnstap.h
index 9a13525bf31..14e042c3c61 100644
--- a/lib/dns/include/dns/dnstap.h
+++ b/lib/dns/include/dns/dnstap.h
@@ -26,13 +26,7 @@
 #include <inttypes.h>
 #include <stdbool.h>
 
-#ifdef HAVE_DNSTAP
-#include <fstrm.h>
-
-#include <protobuf-c/protobuf-c.h>
-#else  /* ifdef HAVE_DNSTAP */
 struct fstrm_iothr_options;
-#endif /* HAVE_DNSTAP */
 
 #include <isc/log.h>
 #include <isc/refcount.h>
diff --git a/lib/dns/tests/dnstap_test.c b/lib/dns/tests/dnstap_test.c
index b41cd14fd45..5c46eda56d9 100644
--- a/lib/dns/tests/dnstap_test.c
+++ b/lib/dns/tests/dnstap_test.c
@@ -37,6 +37,8 @@
 
 #ifdef HAVE_DNSTAP
 
+#include <fstrm.h>
+
 #include <protobuf-c/protobuf-c.h>
 
 #define TAPFILE "testdata/dnstap/dnstap.file"