[NETFILTER]: xt_policy: use the new union nf_inet_addr
authorJan Engelhardt <jengelh@computergmbh.de>
Tue, 15 Jan 2008 07:42:06 +0000 (23:42 -0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 28 Jan 2008 23:02:25 +0000 (15:02 -0800)
Signed-off-by: Jan Engelhardt <jengelh@computergmbh.de>
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netfilter/xt_policy.h
net/netfilter/xt_policy.c

index 45654d359a68c39653af02d832e2f7b2a2eec3de..053d8cc6546401988b13d8fb613d818a7a913deb 100644 (file)
@@ -27,18 +27,33 @@ struct xt_policy_spec
                        reqid:1;
 };
 
+#ifndef __KERNEL__
 union xt_policy_addr
 {
        struct in_addr  a4;
        struct in6_addr a6;
 };
+#endif
 
 struct xt_policy_elem
 {
-       union xt_policy_addr    saddr;
-       union xt_policy_addr    smask;
-       union xt_policy_addr    daddr;
-       union xt_policy_addr    dmask;
+       union {
+#ifdef __KERNEL__
+               struct {
+                       union nf_inet_addr saddr;
+                       union nf_inet_addr smask;
+                       union nf_inet_addr daddr;
+                       union nf_inet_addr dmask;
+               };
+#else
+               struct {
+                       union xt_policy_addr saddr;
+                       union xt_policy_addr smask;
+                       union xt_policy_addr daddr;
+                       union xt_policy_addr dmask;
+               };
+#endif
+       };
        __be32                  spi;
        u_int32_t               reqid;
        u_int8_t                proto;
index 45731ca15b7d55f124c780328766afaea9813d1b..47c2e4328a1b96ca92ffdd74782f66e71e1f2060 100644 (file)
@@ -13,6 +13,7 @@
 #include <linux/init.h>
 #include <net/xfrm.h>
 
+#include <linux/netfilter.h>
 #include <linux/netfilter/xt_policy.h>
 #include <linux/netfilter/x_tables.h>
 
@@ -21,14 +22,14 @@ MODULE_DESCRIPTION("Xtables IPsec policy matching module");
 MODULE_LICENSE("GPL");
 
 static inline bool
-xt_addr_cmp(const union xt_policy_addr *a1, const union xt_policy_addr *m,
-           const union xt_policy_addr *a2, unsigned short family)
+xt_addr_cmp(const union nf_inet_addr *a1, const union nf_inet_addr *m,
+           const union nf_inet_addr *a2, unsigned short family)
 {
        switch (family) {
        case AF_INET:
-               return !((a1->a4.s_addr ^ a2->a4.s_addr) & m->a4.s_addr);
+               return ((a1->ip ^ a2->ip) & m->ip) == 0;
        case AF_INET6:
-               return !ipv6_masked_addr_cmp(&a1->a6, &m->a6, &a2->a6);
+               return ipv6_masked_addr_cmp(&a1->in6, &m->in6, &a2->in6) == 0;
        }
        return false;
 }
@@ -38,12 +39,12 @@ match_xfrm_state(const struct xfrm_state *x, const struct xt_policy_elem *e,
                 unsigned short family)
 {
 #define MATCH_ADDR(x,y,z)      (!e->match.x ||                        \
-                                (xt_addr_cmp(&e->x, &e->y, (z), family) \
+                                (xt_addr_cmp(&e->x, &e->y, (const union nf_inet_addr *)(z), family) \
                                  ^ e->invert.x))
 #define MATCH(x,y)             (!e->match.x || ((e->x == (y)) ^ e->invert.x))
 
-       return MATCH_ADDR(saddr, smask, (union xt_policy_addr *)&x->props.saddr) &&
-              MATCH_ADDR(daddr, dmask, (union xt_policy_addr *)&x->id.daddr) &&
+       return MATCH_ADDR(saddr, smask, &x->props.saddr) &&
+              MATCH_ADDR(daddr, dmask, &x->id.daddr) &&
               MATCH(proto, x->id.proto) &&
               MATCH(mode, x->props.mode) &&
               MATCH(spi, x->id.spi) &&