[NETFILTER]: nf_conntrack: fix helper structure alignment
authorPatrick McHardy <kaber@trash.net>
Sun, 3 Dec 2006 06:04:50 +0000 (22:04 -0800)
committerDavid S. Miller <davem@davemloft.net>
Sun, 3 Dec 2006 06:04:50 +0000 (22:04 -0800)
Adding the alignment to the size doesn't make any sense, what it
should do is align the size of the conntrack structure to the
alignment requirements of the helper structure and return an
aligned pointer in nfct_help().

Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/netfilter/nf_conntrack.h
net/netfilter/nf_conntrack_helper.c

index d0d0e64914485c2fadc340ee87f8bccaa1911ce7..b4beb8c799e565c9b02530d9c3fe951db13c9c25 100644 (file)
@@ -273,6 +273,7 @@ static inline struct nf_conn_help *nfct_help(const struct nf_conn *ct)
        if (!(ct->features & NF_CT_F_HELP))
                return NULL;
 
+       offset = ALIGN(offset, __alignof__(struct nf_conn_help));
        return (struct nf_conn_help *) ((void *)ct + offset);
 }
 
index 03b3ed8fca9e7628027c1158fe08d2f6c0d770b1..2628f4ba35ee806f37d1c2740431299bf46865d7 100644 (file)
@@ -97,13 +97,14 @@ static inline int unhelp(struct nf_conntrack_tuple_hash *i,
 
 int nf_conntrack_helper_register(struct nf_conntrack_helper *me)
 {
-       int ret;
+       int size, ret;
+
        BUG_ON(me->timeout == 0);
 
+       size = ALIGN(sizeof(struct nf_conn), __alignof__(struct nf_conn_help)) +
+              sizeof(struct nf_conn_help);
        ret = nf_conntrack_register_cache(NF_CT_F_HELP, "nf_conntrack:help",
-                                         sizeof(struct nf_conn)
-                                         + sizeof(struct nf_conn_help)
-                                         + __alignof__(struct nf_conn_help));
+                                         size);
        if (ret < 0) {
                printk(KERN_ERR "nf_conntrack_helper_register: Unable to create slab cache for conntracks\n");
                return ret;