netfilter: conntrack: make all extensions 8-byte alignned
authorFlorian Westphal <fw@strlen.de>
Thu, 20 Jan 2022 12:06:59 +0000 (13:06 +0100)
committerPablo Neira Ayuso <pablo@netfilter.org>
Fri, 4 Feb 2022 05:30:28 +0000 (06:30 +0100)
All extensions except one need 8 byte alignment, so just make that the
default.

Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
12 files changed:
include/net/netfilter/nf_conntrack_extend.h
net/netfilter/nf_conntrack_acct.c
net/netfilter/nf_conntrack_ecache.c
net/netfilter/nf_conntrack_extend.c
net/netfilter/nf_conntrack_helper.c
net/netfilter/nf_conntrack_labels.c
net/netfilter/nf_conntrack_seqadj.c
net/netfilter/nf_conntrack_timeout.c
net/netfilter/nf_conntrack_timestamp.c
net/netfilter/nf_nat_core.c
net/netfilter/nf_synproxy_core.c
net/sched/act_ct.c

index c7515d82ab064571bc37d50a42be03c4c8cd6b6a..705a4487f023326c7f37d1cc1f1e2e7391481f55 100644 (file)
@@ -49,7 +49,7 @@ enum nf_ct_ext_id {
 struct nf_ct_ext {
        u8 offset[NF_CT_EXT_NUM];
        u8 len;
-       char data[];
+       char data[] __aligned(8);
 };
 
 static inline bool __nf_ct_ext_exist(const struct nf_ct_ext *ext, u8 id)
@@ -83,10 +83,7 @@ struct nf_ct_ext_type {
        void (*destroy)(struct nf_conn *ct);
 
        enum nf_ct_ext_id id;
-
-       /* Length and min alignment. */
        u8 len;
-       u8 align;
 };
 
 int nf_ct_extend_register(const struct nf_ct_ext_type *type);
index 91bc8df3e4b05e1b4db4fb049d386d8401a217d6..c9b20b86711ca24d56d8fe20da0643c565ab83c9 100644 (file)
@@ -24,7 +24,6 @@ MODULE_PARM_DESC(acct, "Enable connection tracking flow accounting.");
 
 static const struct nf_ct_ext_type acct_extend = {
        .len    = sizeof(struct nf_conn_acct),
-       .align  = __alignof__(struct nf_conn_acct),
        .id     = NF_CT_EXT_ACCT,
 };
 
index 41768ff19464b2bf65e2ea1c3812155a7821cc22..1cf2c8cd6a4a695df747937edac6a4b358a36ab7 100644 (file)
@@ -306,7 +306,6 @@ static int nf_ct_events __read_mostly = NF_CT_EVENTS_DEFAULT;
 
 static const struct nf_ct_ext_type event_extend = {
        .len    = sizeof(struct nf_conntrack_ecache),
-       .align  = __alignof__(struct nf_conntrack_ecache),
        .id     = NF_CT_EXT_ECACHE,
 };
 
index 3dbe2329c3f173a871bad9b6a4154683b02dc572..c62f477c6533b401fcb18bf09e6dc93a50fb9579 100644 (file)
@@ -65,7 +65,7 @@ void *nf_ct_ext_add(struct nf_conn *ct, enum nf_ct_ext_id id, gfp_t gfp)
                return NULL;
        }
 
-       newoff = ALIGN(oldlen, t->align);
+       newoff = ALIGN(oldlen, __alignof__(struct nf_ct_ext));
        newlen = newoff + t->len;
        rcu_read_unlock();
 
index ae4488a13c70cb9196ae0d7f0117fafb7c602389..e8f6a389bd017c0f2e7cc2317875f7db5ffa1feb 100644 (file)
@@ -552,7 +552,6 @@ EXPORT_SYMBOL_GPL(nf_nat_helper_unregister);
 
 static const struct nf_ct_ext_type helper_extend = {
        .len    = sizeof(struct nf_conn_help),
-       .align  = __alignof__(struct nf_conn_help),
        .id     = NF_CT_EXT_HELPER,
 };
 
index 5227925566322fb99a3695e0de3889d9065707bf..6323358dbe73d8e9097d9f517cc775e3b56363e8 100644 (file)
@@ -81,7 +81,6 @@ EXPORT_SYMBOL_GPL(nf_connlabels_put);
 
 static const struct nf_ct_ext_type labels_extend = {
        .len    = sizeof(struct nf_conn_labels),
-       .align  = __alignof__(struct nf_conn_labels),
        .id     = NF_CT_EXT_LABELS,
 };
 
index 3066449f8bd8a3e9a6ae2b302891fd9acf730af8..b13b3a8a1082ac2e77ef31ef78fec72a4c95d7f1 100644 (file)
@@ -235,7 +235,6 @@ EXPORT_SYMBOL_GPL(nf_ct_seq_offset);
 
 static const struct nf_ct_ext_type nf_ct_seqadj_extend = {
        .len    = sizeof(struct nf_conn_seqadj),
-       .align  = __alignof__(struct nf_conn_seqadj),
        .id     = NF_CT_EXT_SEQADJ,
 };
 
index 14387e0b80088e827779e586699870f43801f723..816fe680375d6882cba77afd9f182e54039aa649 100644 (file)
@@ -137,7 +137,6 @@ EXPORT_SYMBOL_GPL(nf_ct_destroy_timeout);
 
 static const struct nf_ct_ext_type timeout_extend = {
        .len    = sizeof(struct nf_conn_timeout),
-       .align  = __alignof__(struct nf_conn_timeout),
        .id     = NF_CT_EXT_TIMEOUT,
 };
 
index f656d393fa92d9a74ec8ad90b5fa941547c1d6ac..81878d9786ba6d469ddfa2ed3e661db91eec0d2c 100644 (file)
@@ -21,7 +21,6 @@ MODULE_PARM_DESC(tstamp, "Enable connection tracking flow timestamping.");
 
 static const struct nf_ct_ext_type tstamp_extend = {
        .len    = sizeof(struct nf_conn_tstamp),
-       .align  = __alignof__(struct nf_conn_tstamp),
        .id     = NF_CT_EXT_TSTAMP,
 };
 
index 2d06a66899b26e275613ab2b0471d81af3f3db38..d0000f63b0af99052894970a8d1f7a32864dbb84 100644 (file)
@@ -877,7 +877,6 @@ static void nf_nat_cleanup_conntrack(struct nf_conn *ct)
 
 static struct nf_ct_ext_type nat_extend __read_mostly = {
        .len            = sizeof(struct nf_conn_nat),
-       .align          = __alignof__(struct nf_conn_nat),
        .destroy        = nf_nat_cleanup_conntrack,
        .id             = NF_CT_EXT_NAT,
 };
index 2dfc5dae065638bbe68a539497509c833f5b32fc..d5c1e93c4ba3dcbb4da05d6a5daa9c8882729f1b 100644 (file)
@@ -238,7 +238,6 @@ synproxy_tstamp_adjust(struct sk_buff *skb, unsigned int protoff,
 
 static struct nf_ct_ext_type nf_ct_synproxy_extend __read_mostly = {
        .len            = sizeof(struct nf_conn_synproxy),
-       .align          = __alignof__(struct nf_conn_synproxy),
        .id             = NF_CT_EXT_SYNPROXY,
 };
 
index f99247fc64682c2bcfa2e5671c2b8bc4ca637511..78ccd16be05e02e0dc08c8b9782e2a7ac08f8230 100644 (file)
@@ -59,7 +59,6 @@ static const struct rhashtable_params zones_params = {
 
 static struct nf_ct_ext_type act_ct_extend __read_mostly = {
        .len            = sizeof(struct nf_conn_act_ct_ext),
-       .align          = __alignof__(struct nf_conn_act_ct_ext),
        .id             = NF_CT_EXT_ACT_CT,
 };