From 58dc37e5b343bd3d04bd7464ec08e50a5641d1c0 Mon Sep 17 00:00:00 2001
From: "Lars H. Rohwedder" <RokerHRO@users.noreply.github.com>
Date: Sun, 3 May 2026 21:35:03 +0200
Subject: [PATCH] move #includes from headers to .c files, remove unnecessary
 #includes.

---
 buffer.c     |  2 ++
 buffer.h     |  2 +-
 cliserv.c    |  1 +
 cliserv.h    |  4 ++--
 nbd-client.c | 11 ++++++-----
 nbd-server.c | 17 ++++++++---------
 nbd-trdump.c |  2 +-
 nbd-trplay.c |  3 ++-
 nbd.h        |  2 +-
 nbdclt.h     |  5 ++++-
 nbdsrv.h     |  3 ++-
 treefiles.c  |  3 ++-
 12 files changed, 32 insertions(+), 23 deletions(-)

diff --git a/buffer.c b/buffer.c
index 7009615d..913f1d1b 100644
--- a/buffer.c
+++ b/buffer.c
@@ -27,6 +27,8 @@ OTHER DEALINGS IN THE SOFTWARE.
 #include <sys/types.h>
 
 #include "buffer.h"
+#include <stdlib.h>
+
 
 struct buffer
 {
diff --git a/buffer.h b/buffer.h
index c92b9a68..328abef0 100644
--- a/buffer.h
+++ b/buffer.h
@@ -25,7 +25,7 @@ OTHER DEALINGS IN THE SOFTWARE.
 #ifndef __TLSPROXY_BUFFERS_H
 #define __TLSPROXY_BUFFERS_H
 
-#include <stdlib.h>
+//#include <stdlib.h>
 #include <sys/types.h>
 
 typedef struct buffer buffer_t;
diff --git a/cliserv.c b/cliserv.c
index 651145f8..795a7560 100644
--- a/cliserv.c
+++ b/cliserv.c
@@ -1,4 +1,5 @@
 #include <config.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <stdio.h>
 #include <syslog.h>
diff --git a/cliserv.h b/cliserv.h
index 9e2d8c01..bec0b5ca 100644
--- a/cliserv.h
+++ b/cliserv.h
@@ -9,14 +9,14 @@
    Send 128 bytes of zeros (reserved for future use)
  */
 
-#include <errno.h>
+// #include <errno.h>
 #include <string.h>
 #include <netdb.h>
 #include <netinet/tcp.h>
 #include <netinet/in.h>
 #include <stdlib.h>
 
-#include "nbd.h"
+//#include "nbd.h"
 
 #ifndef HAVE_FDATASYNC
 #define fdatasync(arg) fsync(arg)
diff --git a/nbd-client.c b/nbd-client.c
index 2e98ba53..f08140b7 100644
--- a/nbd-client.c
+++ b/nbd-client.c
@@ -19,17 +19,18 @@
  */
 
 #include "config.h"
-#include "lfs.h"
+// #include "lfs.h"
+#include "nbd.h"
 
 #include <sys/ioctl.h>
 #include <sys/socket.h>
 #include <sys/un.h>
 #include <sys/types.h>
 #include <unistd.h>
-#include <netinet/tcp.h>
+// #include <netinet/tcp.h>
 #include <netinet/in.h>
 #include <netdb.h>
-#include "netdb-compat.h"
+// #include "netdb-compat.h"
 #include <inttypes.h>
 #include <stdio.h>
 #include <fcntl.h>
@@ -39,12 +40,12 @@
 #include <sys/mman.h>
 #include <signal.h>
 #include <errno.h>
-#include <getopt.h>
+// #include <getopt.h>
 #include <stdarg.h>
 #include <stdbool.h>
 #include <time.h>
 
-#include <linux/ioctl.h>
+// #include <linux/ioctl.h>
 
 #if HAVE_NETLINK
 #include "nbd-netlink.h"
diff --git a/nbd-server.c b/nbd-server.c
index 6a87d445..78f20b5b 100644
--- a/nbd-server.c
+++ b/nbd-server.c
@@ -78,11 +78,11 @@
 #ifdef HAVE_SYS_UIO_H
 #include <sys/uio.h>
 #endif
-#include <sys/param.h>
+// #include <sys/param.h>
 #include <signal.h>
 #include <errno.h>
 #include <libgen.h>
-#include <netinet/tcp.h>
+// #include <netinet/tcp.h>
 #include <netinet/in.h>
 #include <netdb.h>
 #include <syslog.h>
@@ -98,8 +98,8 @@
 #if HAVE_BLKDISCARD
 #include <linux/fs.h>
 #endif
-#include <arpa/inet.h>
-#include <strings.h>
+// #include <arpa/inet.h>
+// #include <strings.h>
 #include <dirent.h>
 #ifdef HAVE_SYS_DIR_H
 #include <sys/dir.h>
@@ -111,7 +111,7 @@
 #include <pwd.h>
 #include <grp.h>
 #include <dirent.h>
-#include <ctype.h>
+// #include <ctype.h>
 #include <inttypes.h>
 
 #include <glib.h>
@@ -120,7 +120,7 @@
 #define MY_NAME "nbd_server"
 #include "cliserv.h"
 #include "nbd-debug.h"
-#include "netdb-compat.h"
+// #include "netdb-compat.h"
 #include "backend.h"
 #include "treefiles.h"
 #include "nbd-helper.h"
@@ -379,9 +379,8 @@ static void socket_read(CLIENT* client, void *buf, size_t len) {
  * @param bufsiz the size of the buffer
  **/
 static inline void consume(CLIENT* c, size_t len, void * buf, size_t bufsiz) {
-	size_t curlen;
 	while (len>0) {
-		curlen = (len>bufsiz)?bufsiz:len;
+        const size_t curlen = (len>bufsiz)?bufsiz:len;
 		socket_read(c, buf, curlen);
 		len -= curlen;
 	}
@@ -555,7 +554,7 @@ SERVER* cmdline(int argc, char *argv[], struct generic_conf *genconf) {
 	int i=0;
 	int nonspecial=0;
 	int c;
-	struct option long_options[] = {
+    const struct option long_options[] = {
 		{"read-only", no_argument, NULL, 'r'},
 		{"multi-file", no_argument, NULL, 'm'},
 		{"copy-on-write", no_argument, NULL, 'c'},
diff --git a/nbd-trdump.c b/nbd-trdump.c
index eb02245d..6a6d8513 100644
--- a/nbd-trdump.c
+++ b/nbd-trdump.c
@@ -8,7 +8,7 @@
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
-#include <sys/time.h>
+// #include <sys/time.h>
 #include <sys/types.h>
 #include <stdint.h>
 #include <stdbool.h>
diff --git a/nbd-trplay.c b/nbd-trplay.c
index deeea517..8c5f181f 100644
--- a/nbd-trplay.c
+++ b/nbd-trplay.c
@@ -9,10 +9,11 @@
  */
 
 #include <stdlib.h>
+#include <errno.h>
 #include <limits.h>
 #include <stdio.h>
 #include <string.h>
-#include <sys/time.h>
+// #include <sys/time.h>
 #include <sys/types.h>
 #include <fcntl.h>
 #include <stdint.h>
diff --git a/nbd.h b/nbd.h
index 049e7455..348aca7c 100644
--- a/nbd.h
+++ b/nbd.h
@@ -15,7 +15,7 @@
 #ifndef LINUX_NBD_H
 #define LINUX_NBD_H
 
-//#include <linux/types.h>
+#include <stdint.h>
 
 #define NBD_SET_SOCK	_IO( 0xab, 0 )
 #define NBD_SET_BLKSIZE	_IO( 0xab, 1 )
diff --git a/nbdclt.h b/nbdclt.h
index 51e7ccb6..e16cddee 100644
--- a/nbdclt.h
+++ b/nbdclt.h
@@ -1,6 +1,9 @@
 #ifndef NBDCLT_H
 #define NBDCLT_H
 
+#include <stdint.h>
+#include <stdbool.h>
+
 typedef struct {
 	char *name;
 	char *dev;
@@ -23,7 +26,7 @@ typedef struct {
 	bool preinit;
 	bool force_ro;
 	bool tls;
-        bool persist_mode;
+    bool persist_mode;
 	char *priority;
 	int dead_conn_timeout;
 } CLIENT;
diff --git a/nbdsrv.h b/nbdsrv.h
index 4590c862..2e0df63f 100644
--- a/nbdsrv.h
+++ b/nbdsrv.h
@@ -1,11 +1,12 @@
 #ifndef NBDSRV_H
 #define NBDSRV_H
 
-#include "lfs.h"
+// #include "lfs.h"
 
 #include <glib.h>
 #include <stdbool.h>
 #include <stdint.h>
+#include <pthread.h>
 
 #include <sys/socket.h>
 #include <sys/types.h>
diff --git a/treefiles.c b/treefiles.c
index a5552226..9814b8ab 100644
--- a/treefiles.c
+++ b/treefiles.c
@@ -1,10 +1,11 @@
+#include <errno.h>
 #include <fcntl.h>
 #include <limits.h> // for PATH_MAX
 #include <inttypes.h>
 #include <stdio.h>
 #include <unistd.h>
 #include <sys/stat.h>
-#include <sys/types.h>
+// #include <sys/types.h>
 
 #include "config.h"
 #include "cliserv.h"