Improve include directives

Update #include directives by removing such that aren't needed, and add such that should be there, since there's a direct dependencies to symbols or macros in them. Signed-off-by: Mattias Rönnblom <[email protected]>
Ericsson · Mar 6, 2022 · 4fd8335 · 4fd8335
1 parent 3ba4cbb
commit 4fd8335
Show file tree

Hide file tree

Showing 33 changed files with 78 additions and 50 deletions.
diff --git a/include/xcm.h b/include/xcm.h
@@ -1260,6 +1260,7 @@ extern "C" {
 
 #include <errno.h>
 #include <stdbool.h>
+#include <stddef.h>
 #include <sys/types.h>
 #include <xcm_attr_map.h>
 

diff --git a/include/xcm_attr_map.h b/include/xcm_attr_map.h
@@ -22,8 +22,8 @@ extern "C" {
 #include <xcm_attr_types.h>
 
 #include <stdbool.h>
+#include <stddef.h>
 #include <stdint.h>
-#include <sys/types.h>
 
 struct xcm_attr_map;
 

diff --git a/libxcm/active_fd.c b/libxcm/active_fd.c
@@ -4,8 +4,10 @@
 #include "util.h"
 
 #include <pthread.h>
+#include <stddef.h>
 #include <sys/eventfd.h>
 #include <sys/queue.h>
+#include <unistd.h>
 
 /*
  * The Linux kernel limits the number of epoll instances fds the

diff --git a/libxcm/common_tp.c b/libxcm/common_tp.c
@@ -5,9 +5,15 @@
 
 #include "common_tp.h"
 
+#include "log_tp.h"
 #include "util.h"
+#include "xcm.h"
+#include "xcm_addr.h"
 
+#include <netinet/in.h>
 #include <stdlib.h>
+#include <string.h>
+#include <sys/socket.h>
 
 void tp_ip_to_sockaddr(const struct xcm_addr_ip *xcm_ip, uint16_t port,
 		       int64_t scope, struct sockaddr *sockaddr)

diff --git a/libxcm/common_tp.h b/libxcm/common_tp.h
@@ -6,9 +6,15 @@
 #ifndef COMMON_TP_H
 #define COMMON_TP_H
 
-#include "log_tp.h"
 #include "xcm_addr.h"
 
+#include <errno.h>
+#include <stdbool.h>
+#include <stddef.h>
+#include <stdint.h>
+
+struct xcm_socket;
+
 void tp_ip_to_sockaddr(const struct xcm_addr_ip *xcm_ip, uint16_t port,
 		       int64_t scope, struct sockaddr *sockaddr);
 

diff --git a/libxcm/ctl.c b/libxcm/ctl.c
@@ -14,12 +14,12 @@
 #include "xcm_attr.h"
 #include "xcm_tp.h"
 
-#include <assert.h>
 #include <linux/un.h>
 #include <stdio.h>
+#include <string.h>
+#include <sys/epoll.h>
 #include <sys/socket.h>
 #include <sys/stat.h>
-#include <sys/types.h>
 #include <unistd.h>
 
 #define MAX_CLIENTS (2)

diff --git a/libxcm/ctl.h b/libxcm/ctl.h
@@ -13,7 +13,9 @@
  * from outside the process using XCM.
  */
 
-#include "xcm.h"
+#include <stdbool.h>
+
+struct xcm_socket;
 
 struct ctl;
 

diff --git a/libxcm/ctx_store.c b/libxcm/ctx_store.c
@@ -3,13 +3,19 @@
 #include "log_tls.h"
 #include "util.h"
 
-#include <openssl/x509_vfy.h>
-#include <openssl/x509v3.h>
+#include <openssl/sha.h>
+#include <openssl/ssl.h>
+#include <openssl/ssl3.h>
+#include <openssl/x509.h>
+
 #include <pthread.h>
+#include <stdbool.h>
 #include <stdint.h>
+#include <string.h>
 #include <sys/queue.h>
 #include <sys/stat.h>
 #include <sys/types.h>
+#include <time.h>
 
 /* OpenSSL 1.1.0 lacks TLS 1.3 support */
 #ifdef SSL_OP_NO_TLSv1_3

diff --git a/libxcm/ctx_store.h b/libxcm/ctx_store.h
@@ -1,8 +1,7 @@
 #ifndef CTX_STORE
 #define CTX_STORE
 
-#include <openssl/ssl.h>
-#include <stdbool.h>
+#include <openssl/ossl_typ.h>
 
 void ctx_store_init(void);
 

diff --git a/libxcm/log.c b/libxcm/log.c
@@ -11,6 +11,7 @@
 #include <inttypes.h>
 #include <libgen.h>
 #include <limits.h>
+#include <stdarg.h>
 #include <stdio.h>
 #include <string.h>
 #include <unistd.h>

diff --git a/libxcm/log.h b/libxcm/log.h
@@ -6,11 +6,10 @@
 #ifndef LOG_H
 #define LOG_H
 
-#include "xcm.h"
-
-#include <stdarg.h>
 #include <stdbool.h>
 
+struct xcm_socket;
+
 #define log_event(type, sock, ...)                                      \
     do {                                                                \
 	if (log_is_enabled(type))                                       \

diff --git a/libxcm/log_ctl.h b/libxcm/log_ctl.h
@@ -6,7 +6,6 @@
 #ifndef LOG_CTL_H
 #define LOG_CTL_H
 
-#include "common_tp.h"
 #include "log.h"
 
 #define LOG_RUN_STAT_ERROR(s, path, reason_errno)	       \

diff --git a/libxcm/log_tls.c b/libxcm/log_tls.c
@@ -8,6 +8,7 @@
 #include "util.h"
 
 #include <openssl/err.h>
+#include <stdio.h>
 #include <string.h>
 
 void hash_description(uint8_t *hash, size_t hash_len, char *buf)

diff --git a/libxcm/log_tls.h b/libxcm/log_tls.h
@@ -8,6 +8,7 @@
 
 #include "log.h"
 
+#include <stddef.h>
 #include <stdint.h>
 
 #define LOG_TLS_HANDSHAKE(s, client)					\

diff --git a/libxcm/log_tp.c b/libxcm/log_tp.c
@@ -1,8 +1,15 @@
 #include "log_tp.h"
 
 #include "util.h"
+#include "xcm_attr_map.h"
 
+#include <arpa/inet.h>
 #include <inttypes.h>
+#include <netinet/in.h>
+#include <stdbool.h>
+#include <stdint.h>
+#include <string.h>
+#include <sys/socket.h>
 
 const char *log_ip_str(sa_family_t family, const void *ip)
 {

diff --git a/libxcm/log_tp.h b/libxcm/log_tp.h
@@ -7,15 +7,12 @@
 #define LOG_SOCK_H
 
 #include "log.h"
+#include "xcm_addr.h"
+#include "xcm_attr_map.h"
 #include "xcm_attr_types.h"
 
-#include <arpa/inet.h>
 #include <inttypes.h>
-#include <netinet/in.h>
 #include <stdio.h>
-#include <string.h>
-#include <sys/socket.h>
-#include <xcm_attr_map.h>
 
 #include "common_tp.h"
 

diff --git a/libxcm/mbuf.h b/libxcm/mbuf.h
@@ -12,8 +12,9 @@
 #include "util.h"
 
 #include <assert.h>
-#include <inttypes.h>
+#include <netinet/in.h>
 #include <stdbool.h>
+#include <stdint.h>
 #include <stdlib.h>
 #include <string.h>
 

diff --git a/libxcm/tcp_attr.c b/libxcm/tcp_attr.c
@@ -8,7 +8,6 @@
 #include "log_tp.h"
 #include "util.h"
 
-#include <arpa/inet.h>
 #include <errno.h>
 #include <limits.h>
 #include <netinet/in.h>

diff --git a/libxcm/tcp_attr.h b/libxcm/tcp_attr.h
@@ -6,10 +6,8 @@
 #ifndef TCP_ATTR_H
 #define TCP_ATTR_H
 
-#include "xcm_tp.h"
-
 #include <stdbool.h>
-#include <sys/types.h>
+#include <stdint.h>
 
 #define XCM_TCP_KEEPALIVE (true)
 #define XCM_TCP_KEEPALIVE_TIME (1)

diff --git a/libxcm/xcm.c b/libxcm/xcm.c
@@ -7,19 +7,16 @@
 
 #include "log_epoll.h"
 #include "util.h"
-#include "xcm_addr.h"
 #include "xcm_attr_names.h"
 #include "xcm_tp.h"
 
-#ifdef XCM_CTL
-#include "ctl.h"
-#endif
-
 #include <poll.h>
+#include <stdint.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <unistd.h>
+#include <sys/epoll.h>
 
 #define XCM_ENV_DEBUG "XCM_DEBUG"
 

diff --git a/libxcm/xcm_addr.c b/libxcm/xcm_addr.c
@@ -18,6 +18,7 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
+#include <sys/socket.h>
 
 static bool has_space(const char *s)
 {

diff --git a/libxcm/xcm_addr_compat.c b/libxcm/xcm_addr_compat.c
@@ -3,8 +3,10 @@
  * Copyright(c) 2020 Ericsson AB
  */
 
-#include <assert.h>
+#include <netinet/in.h>
+#include <stdint.h>
 #include <string.h>
+#include <sys/socket.h>
 #include <xcm_addr.h>
 
 static int delegate_parse(int (parse_fun)(const char *, struct xcm_addr_host *,

diff --git a/libxcm/xcm_attr_map.c b/libxcm/xcm_attr_map.c
@@ -2,6 +2,7 @@
 
 #include "util.h"
 
+#include <string.h>
 #include <sys/queue.h>
 
 struct attr

diff --git a/libxcm/xcm_compat.c b/libxcm/xcm_compat.c
@@ -6,9 +6,11 @@
 #include "xcm.h"
 
 #include "common_tp.h"
-#include "util.h"
+#include "log_tp.h"
 #include "xcm_tp.h"
 
+#include <string.h>
+
 int xcm_want(struct xcm_socket *s, int condition, int *fds,
 	     int *events, size_t capacity)
 {

diff --git a/libxcm/xcm_dns.h b/libxcm/xcm_dns.h
@@ -6,10 +6,9 @@
 #ifndef XCM_DNS_H
 #define XCM_DNS_H
 
-#include "xcm_tp.h"
+#include "xcm_addr.h"
 
 #include <stdbool.h>
-#include <xcm_addr.h>
 
 struct xcm_dns_query;
 

diff --git a/libxcm/xcm_dns_glibc.c b/libxcm/xcm_dns_glibc.c
@@ -9,9 +9,16 @@
 #include "epoll_reg.h"
 #include "log_tp.h"
 #include "util.h"
+#include "xcm_addr.h"
 
 #include <netdb.h>
 #include <signal.h>
+#include <netinet/in.h>
+#include <stdbool.h>
+#include <stdlib.h>
+#include <string.h>
+#include <sys/socket.h>
+#include <unistd.h>
 
 enum query_state {
     query_state_resolving,

diff --git a/libxcm/xcm_tp.h b/libxcm/xcm_tp.h
@@ -6,8 +6,6 @@
 #ifndef XCM_TP_H
 #define XCM_TP_H
 
-#include <sys/types.h>
-
 #include "config.h"
 #include "xcm.h"
 #include "xcm_addr_limits.h"

diff --git a/libxcm/xcm_tp_btls.c b/libxcm/xcm_tp_btls.c
@@ -19,14 +19,20 @@
 #include "xcm_dns.h"
 #include "xcm_tp.h"
 
-#include <arpa/inet.h>
+#include <limits.h>
+#include <netinet/in.h>
+#include <openssl/asn1.h>
 #include <openssl/err.h>
-#include <openssl/ssl.h>
+#include <openssl/obj_mac.h>
+#include <openssl/ossl_typ.h>
+#include <openssl/safestack.h>
+#include <openssl/ssl3.h>
+#include <openssl/x509.h>
 #include <openssl/x509v3.h>
 #include <signal.h>
-#include <stdarg.h>
 #include <stdlib.h>
 #include <string.h>
+#include <sys/socket.h>
 #include <unistd.h>
 
 /*

diff --git a/libxcm/xcm_tp_tcp.c b/libxcm/xcm_tp_tcp.c
@@ -17,9 +17,7 @@
 #include "xcm_dns.h"
 #include "xcm_tp.h"
 
-#include <arpa/inet.h>
 #include <netinet/in.h>
-#include <netinet/tcp.h>
 #include <stdlib.h>
 #include <string.h>
 #include <sys/socket.h>

diff --git a/libxcm/xcm_tp_tls.c b/libxcm/xcm_tp_tls.c
@@ -4,7 +4,6 @@
  */
 
 #include "common_tp.h"
-#include "log_tls.h"
 #include "log_tp.h"
 #include "mbuf.h"
 #include "util.h"
@@ -15,7 +14,6 @@
 
 #include <stdlib.h>
 #include <string.h>
-#include <unistd.h>
 
 /*
  * TLS XCM Transport