| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <4cf2e26b-2727-4b50-9ada-56a6be814dca@freemail.hu>
Date: Tue, 7 Jan 2025 22:59:21 +0100
From: Szőke Benjamin <egyszeregy@...email.hu>
To: Jozsef Kadlecsik <kadlec@...filter.org>
Cc: fw@...len.de, pablo@...filter.org, lorenzo@...nel.org,
daniel@...earbox.net, leitao@...ian.org, amiculas@...co.com,
davem@...emloft.net, dsahern@...nel.org, edumazet@...gle.com,
kuba@...nel.org, pabeni@...hat.com, horms@...nel.org,
netfilter-devel@...r.kernel.org, coreteam@...filter.org,
linux-kernel@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH 08/10] netfilter: Adjust code style of xt_*.h, ipt_*.h
files.
2025. 01. 07. 20:39 keltezéssel, Jozsef Kadlecsik írta:
> On Tue, 7 Jan 2025, egyszeregy@...email.hu wrote:
>
>> From: Benjamin Szőke <egyszeregy@...email.hu>
>>
>> - Adjust tab indents
>> - Fix format of #define macros
>
> I don't really understand why it'd be important to use parentheses around
> plain constant values in macros. The kernel coding style does not list it
> as a requirement, see 12) 4. in Documentation/process/coding-style.rst.
>
If it would be more than just a const value, parentheses is a must have thing
for it (now for it, it is not critical to have it but better to get used to
this). This is how my hand automatically do it, to avoid the syntax problem in
this coding.
> Best regards,
> Jozsef
>
>> Signed-off-by: Benjamin Szőke <egyszeregy@...email.hu>
>> ---
>> include/uapi/linux/netfilter/xt_dscp.h | 6 +++---
>> include/uapi/linux/netfilter/xt_rateest.h | 4 ++--
>> include/uapi/linux/netfilter/xt_tcpmss.h | 6 +++---
>> include/uapi/linux/netfilter_ipv4/ipt_ecn.h | 8 ++++----
>> include/uapi/linux/netfilter_ipv4/ipt_ttl.h | 3 +--
>> include/uapi/linux/netfilter_ipv6/ip6t_hl.h | 3 +--
>> 6 files changed, 14 insertions(+), 16 deletions(-)
>>
>> diff --git a/include/uapi/linux/netfilter/xt_dscp.h b/include/uapi/linux/netfilter/xt_dscp.h
>> index bcfe4afa6351..22b6488ef2e7 100644
>> --- a/include/uapi/linux/netfilter/xt_dscp.h
>> +++ b/include/uapi/linux/netfilter/xt_dscp.h
>> @@ -15,9 +15,9 @@
>>
>> #include <linux/types.h>
>>
>> -#define XT_DSCP_MASK 0xfc /* 11111100 */
>> -#define XT_DSCP_SHIFT 2
>> -#define XT_DSCP_MAX 0x3f /* 00111111 */
>> +#define XT_DSCP_MASK (0xfc) /* 11111100 */
>> +#define XT_DSCP_SHIFT (2)
>> +#define XT_DSCP_MAX (0x3f) /* 00111111 */
>>
>> /* match info */
>> struct xt_dscp_info {
>> diff --git a/include/uapi/linux/netfilter/xt_rateest.h b/include/uapi/linux/netfilter/xt_rateest.h
>> index da9727fa527b..f719bd501d1a 100644
>> --- a/include/uapi/linux/netfilter/xt_rateest.h
>> +++ b/include/uapi/linux/netfilter/xt_rateest.h
>> @@ -22,8 +22,8 @@ enum xt_rateest_match_mode {
>> };
>>
>> struct xt_rateest_match_info {
>> - char name1[IFNAMSIZ];
>> - char name2[IFNAMSIZ];
>> + char name1[IFNAMSIZ];
>> + char name2[IFNAMSIZ];
>> __u16 flags;
>> __u16 mode;
>> __u32 bps1;
>> diff --git a/include/uapi/linux/netfilter/xt_tcpmss.h b/include/uapi/linux/netfilter/xt_tcpmss.h
>> index 3ee4acaa6e03..ad858ae93e6a 100644
>> --- a/include/uapi/linux/netfilter/xt_tcpmss.h
>> +++ b/include/uapi/linux/netfilter/xt_tcpmss.h
>> @@ -4,11 +4,11 @@
>>
>> #include <linux/types.h>
>>
>> -#define XT_TCPMSS_CLAMP_PMTU 0xffff
>> +#define XT_TCPMSS_CLAMP_PMTU (0xffff)
>>
>> struct xt_tcpmss_match_info {
>> - __u16 mss_min, mss_max;
>> - __u8 invert;
>> + __u16 mss_min, mss_max;
>> + __u8 invert;
>> };
>>
>> struct xt_tcpmss_info {
>> diff --git a/include/uapi/linux/netfilter_ipv4/ipt_ecn.h b/include/uapi/linux/netfilter_ipv4/ipt_ecn.h
>> index a6d479aece21..0594dd49d13f 100644
>> --- a/include/uapi/linux/netfilter_ipv4/ipt_ecn.h
>> +++ b/include/uapi/linux/netfilter_ipv4/ipt_ecn.h
>> @@ -16,10 +16,10 @@
>>
>> #define ipt_ecn_info xt_ecn_info
>>
>> -#define IPT_ECN_OP_SET_IP 0x01 /* set ECN bits of IPv4 header */
>> -#define IPT_ECN_OP_SET_ECE 0x10 /* set ECE bit of TCP header */
>> -#define IPT_ECN_OP_SET_CWR 0x20 /* set CWR bit of TCP header */
>> -#define IPT_ECN_OP_MASK 0xce
>> +#define IPT_ECN_OP_SET_IP (0x01) /* set ECN bits of IPv4 header */
>> +#define IPT_ECN_OP_SET_ECE (0x10) /* set ECE bit of TCP header */
>> +#define IPT_ECN_OP_SET_CWR (0x20) /* set CWR bit of TCP header */
>> +#define IPT_ECN_OP_MASK (0xce)
>>
>> enum {
>> IPT_ECN_IP_MASK = XT_ECN_IP_MASK,
>> diff --git a/include/uapi/linux/netfilter_ipv4/ipt_ttl.h b/include/uapi/linux/netfilter_ipv4/ipt_ttl.h
>> index c21eb6651353..15c75a4ba355 100644
>> --- a/include/uapi/linux/netfilter_ipv4/ipt_ttl.h
>> +++ b/include/uapi/linux/netfilter_ipv4/ipt_ttl.h
>> @@ -9,13 +9,12 @@
>> #include <linux/types.h>
>>
>> enum {
>> - IPT_TTL_EQ = 0, /* equals */
>> + IPT_TTL_EQ = 0, /* equals */
>> IPT_TTL_NE, /* not equals */
>> IPT_TTL_LT, /* less than */
>> IPT_TTL_GT, /* greater than */
>> };
>>
>> -
>> struct ipt_ttl_info {
>> __u8 mode;
>> __u8 ttl;
>> diff --git a/include/uapi/linux/netfilter_ipv6/ip6t_hl.h b/include/uapi/linux/netfilter_ipv6/ip6t_hl.h
>> index caef38a63b8f..4af05c86dcd5 100644
>> --- a/include/uapi/linux/netfilter_ipv6/ip6t_hl.h
>> +++ b/include/uapi/linux/netfilter_ipv6/ip6t_hl.h
>> @@ -9,13 +9,12 @@
>> #include <linux/types.h>
>>
>> enum {
>> - IP6T_HL_EQ = 0, /* equals */
>> + IP6T_HL_EQ = 0, /* equals */
>> IP6T_HL_NE, /* not equals */
>> IP6T_HL_LT, /* less than */
>> IP6T_HL_GT, /* greater than */
>> };
>>
>> -
>> struct ip6t_hl_info {
>> __u8 mode;
>> __u8 hop_limit;
>> --
>> 2.43.5
>>
>>
>
Powered by blists - more mailing lists