| From f319389525b066b7dc6d389c88f16a0df3b8f189 Mon Sep 17 00:00:00 2001 |
| Message-Id: <f319389525b066b7dc6d389c88f16a0df3b8f189.1656908915.git.baruch@tkos.co.il> |
| From: Nick Hainke <vincent@systemli.org> |
| Date: Mon, 16 May 2022 18:16:41 +0200 |
| Subject: [PATCH] treewide: use uint* instead of u_int* |
| |
| Gcc complains about missing types. Some commits introduced u_int* instead |
| of uint*. Use uint treewide. |
| |
| Fixes errors in the form of: |
| In file included from xtables-legacy-multi.c:5: |
| xshared.h:83:56: error: unknown type name 'u_int16_t'; did you mean 'uint16_t'? |
| 83 | set_option(unsigned int *options, unsigned int option, u_int16_t *invflg, |
| | ^~~~~~~~~ |
| | uint16_t |
| make[6]: *** [Makefile:712: xtables_legacy_multi-xtables-legacy-multi.o] Error 1 |
| |
| Avoid libipq API breakage by adjusting libipq.h include accordingly. For |
| arpt_mangle.h kernel uAPI header, apply same change as in kernel commit |
| e91ded8db5747 ("uapi: netfilter_arp: use __u8 instead of u_int8_t"). |
| |
| Signed-off-by: Nick Hainke <vincent@systemli.org> |
| Signed-off-by: Phil Sutter <phil@nwl.cc> |
| Signed-off-by: Baruch Siach <baruch@tkos.co.il> |
| --- |
| Upstream status: commit f319389525b066b7dc6d389c88f16a0df3b8f189 |
| |
| extensions/libxt_conntrack.c | 2 +- |
| include/libipq/libipq.h | 8 ++++---- |
| include/libiptc/libxtc.h | 2 +- |
| include/linux/netfilter_arp/arpt_mangle.h | 2 +- |
| iptables/xshared.c | 2 +- |
| iptables/xshared.h | 2 +- |
| libipq/ipq_create_handle.3 | 2 +- |
| libipq/ipq_set_mode.3 | 2 +- |
| 8 files changed, 11 insertions(+), 11 deletions(-) |
| |
| diff --git a/extensions/libxt_conntrack.c b/extensions/libxt_conntrack.c |
| index 64018ce152b7..234085c5211c 100644 |
| --- a/extensions/libxt_conntrack.c |
| +++ b/extensions/libxt_conntrack.c |
| @@ -778,7 +778,7 @@ matchinfo_print(const void *ip, const struct xt_entry_match *match, int numeric, |
| |
| static void |
| conntrack_dump_ports(const char *prefix, const char *opt, |
| - u_int16_t port_low, u_int16_t port_high) |
| + uint16_t port_low, uint16_t port_high) |
| { |
| if (port_high == 0 || port_low == port_high) |
| printf(" %s%s %u", prefix, opt, port_low); |
| diff --git a/include/libipq/libipq.h b/include/libipq/libipq.h |
| index 3cd1329217a0..dd0cb205179c 100644 |
| --- a/include/libipq/libipq.h |
| +++ b/include/libipq/libipq.h |
| @@ -24,7 +24,7 @@ |
| #include <errno.h> |
| #include <unistd.h> |
| #include <fcntl.h> |
| -#include <sys/types.h> |
| +#include <stdint.h> |
| #include <sys/socket.h> |
| #include <sys/uio.h> |
| #include <asm/types.h> |
| @@ -48,19 +48,19 @@ typedef unsigned long ipq_id_t; |
| struct ipq_handle |
| { |
| int fd; |
| - u_int8_t blocking; |
| + uint8_t blocking; |
| struct sockaddr_nl local; |
| struct sockaddr_nl peer; |
| }; |
| |
| -struct ipq_handle *ipq_create_handle(u_int32_t flags, u_int32_t protocol); |
| +struct ipq_handle *ipq_create_handle(uint32_t flags, uint32_t protocol); |
| |
| int ipq_destroy_handle(struct ipq_handle *h); |
| |
| ssize_t ipq_read(const struct ipq_handle *h, |
| unsigned char *buf, size_t len, int timeout); |
| |
| -int ipq_set_mode(const struct ipq_handle *h, u_int8_t mode, size_t len); |
| +int ipq_set_mode(const struct ipq_handle *h, uint8_t mode, size_t len); |
| |
| ipq_packet_msg_t *ipq_get_packet(const unsigned char *buf); |
| |
| diff --git a/include/libiptc/libxtc.h b/include/libiptc/libxtc.h |
| index 37010188b64f..a1d16ef996d7 100644 |
| --- a/include/libiptc/libxtc.h |
| +++ b/include/libiptc/libxtc.h |
| @@ -10,7 +10,7 @@ extern "C" { |
| #endif |
| |
| #ifndef XT_MIN_ALIGN |
| -/* xt_entry has pointers and u_int64_t's in it, so if you align to |
| +/* xt_entry has pointers and uint64_t's in it, so if you align to |
| it, you'll also align to any crazy matches and targets someone |
| might write */ |
| #define XT_MIN_ALIGN (__alignof__(struct xt_entry)) |
| diff --git a/include/linux/netfilter_arp/arpt_mangle.h b/include/linux/netfilter_arp/arpt_mangle.h |
| index 250f502902bb..8c2b16a1f5a0 100644 |
| --- a/include/linux/netfilter_arp/arpt_mangle.h |
| +++ b/include/linux/netfilter_arp/arpt_mangle.h |
| @@ -13,7 +13,7 @@ struct arpt_mangle |
| union { |
| struct in_addr tgt_ip; |
| } u_t; |
| - u_int8_t flags; |
| + __u8 flags; |
| int target; |
| }; |
| |
| diff --git a/iptables/xshared.c b/iptables/xshared.c |
| index a8512d380815..9b5e5b5bddc2 100644 |
| --- a/iptables/xshared.c |
| +++ b/iptables/xshared.c |
| @@ -1025,7 +1025,7 @@ static const int inverse_for_options[NUMBER_OF_OPT] = |
| }; |
| |
| void |
| -set_option(unsigned int *options, unsigned int option, u_int16_t *invflg, |
| +set_option(unsigned int *options, unsigned int option, uint16_t *invflg, |
| bool invert) |
| { |
| if (*options & option) |
| diff --git a/iptables/xshared.h b/iptables/xshared.h |
| index 14568bb00fb6..f82129883968 100644 |
| --- a/iptables/xshared.h |
| +++ b/iptables/xshared.h |
| @@ -80,7 +80,7 @@ struct xtables_target; |
| #define IPT_INV_ARPHRD 0x0800 |
| |
| void |
| -set_option(unsigned int *options, unsigned int option, u_int16_t *invflg, |
| +set_option(unsigned int *options, unsigned int option, uint16_t *invflg, |
| bool invert); |
| |
| /** |
| diff --git a/libipq/ipq_create_handle.3 b/libipq/ipq_create_handle.3 |
| index 11ef95c4b79f..ebe46daab92d 100644 |
| --- a/libipq/ipq_create_handle.3 |
| +++ b/libipq/ipq_create_handle.3 |
| @@ -24,7 +24,7 @@ ipq_create_handle, ipq_destroy_handle \(em create and destroy libipq handles. |
| .br |
| .B #include <libipq.h> |
| .sp |
| -.BI "struct ipq_handle *ipq_create_handle(u_int32_t " flags ", u_int32_t " protocol ");" |
| +.BI "struct ipq_handle *ipq_create_handle(uint32_t " flags ", uint32_t " protocol ");" |
| .br |
| .BI "int ipq_destroy_handle(struct ipq_handle *" h ); |
| .SH DESCRIPTION |
| diff --git a/libipq/ipq_set_mode.3 b/libipq/ipq_set_mode.3 |
| index 0edd3c00a84f..e206886c8c62 100644 |
| --- a/libipq/ipq_set_mode.3 |
| +++ b/libipq/ipq_set_mode.3 |
| @@ -24,7 +24,7 @@ ipq_set_mode \(em set the ip_queue queuing mode |
| .br |
| .B #include <libipq.h> |
| .sp |
| -.BI "int ipq_set_mode(const struct ipq_handle *" h ", u_int8_t " mode ", size_t " range ); |
| +.BI "int ipq_set_mode(const struct ipq_handle *" h ", uint8_t " mode ", size_t " range ); |
| .SH DESCRIPTION |
| The |
| .B ipq_set_mode |
| -- |
| 2.35.1 |
| |