netfilter: ipset: whitespace and coding fixes detected by checkpatch.pl

Signed-off-by: Jozsef Kadlecsik <kadlec@blackhole.kfki.hu>
Signed-off-by: Patrick McHardy <kaber@trash.net>

7 files changed, 22 insertions, 21 deletions

diff --git a/net/netfilter/ipset/ip_set_bitmap_ipmac.c b/net/netfilter/ipset/ip_set_bitmap_ipmac.c
index 51ab66435a0a..56096f544978 100644
--- a/net/netfilter/ipset/ip_set_bitmap_ipmac.c
+++ b/net/netfilter/ipset/ip_set_bitmap_ipmac.c
@@ -635,7 +635,8 @@ static struct ip_set_type bitmap_ipmac_type = {
         },
         .adt_policy     = {
                 [IPSET_ATTR_IP]         = { .type = NLA_NESTED },
-                [IPSET_ATTR_ETHER]      = { .type = NLA_BINARY, .len  = ETH_ALEN },
+                [IPSET_ATTR_ETHER]      = { .type = NLA_BINARY,
+                                            .len  = ETH_ALEN },
                 [IPSET_ATTR_TIMEOUT]    = { .type = NLA_U32 },
                 [IPSET_ATTR_LINENO]     = { .type = NLA_U32 },
         },
diff --git a/net/netfilter/ipset/ip_set_core.c b/net/netfilter/ipset/ip_set_core.c
index 80a1262104bf..c012985a5a26 100644
--- a/net/netfilter/ipset/ip_set_core.c
+++ b/net/netfilter/ipset/ip_set_core.c
@@ -683,8 +683,8 @@ ip_set_create(struct sock *ctnl, struct sk_buff *skb,
         if (attr[IPSET_ATTR_DATA] &&
             nla_parse_nested(tb, IPSET_ATTR_CREATE_MAX, attr[IPSET_ATTR_DATA],
                              set->type->create_policy)) {
-                ret = -IPSET_ERR_PROTOCOL;
-                goto put_out;
+                ret = -IPSET_ERR_PROTOCOL;
+                goto put_out;
         }
 
         ret = set->type->create(set, tb, flags);
diff --git a/net/netfilter/ipset/ip_set_hash_net.c b/net/netfilter/ipset/ip_set_hash_net.c
index 050163fb6094..2d4b1f48e8c9 100644
--- a/net/netfilter/ipset/ip_set_hash_net.c
+++ b/net/netfilter/ipset/ip_set_hash_net.c
@@ -187,7 +187,7 @@ hash_net4_uadt(struct ip_set *set, struct nlattr *tb[],
                         return -IPSET_ERR_TIMEOUT;
                 timeout = ip_set_timeout_uget(tb[IPSET_ATTR_TIMEOUT]);
         }
-        
+
         if (adt == IPSET_TEST || !tb[IPSET_ATTR_IP_TO]) {
                 data.ip = htonl(ip & ip_set_hostmask(data.cidr));
                 ret = adtfn(set, &data, timeout, flags);
@@ -205,7 +205,7 @@ hash_net4_uadt(struct ip_set *set, struct nlattr *tb[],
                         return -IPSET_ERR_HASH_RANGE;
         }
         if (retried)
-                ip = h->next.ip;                
+                ip = h->next.ip;
         while (!after(ip, ip_to)) {
                 data.ip = htonl(ip);
                 last = ip_set_range_to_cidr(ip, ip_to, &data.cidr);
diff --git a/net/netfilter/ipset/ip_set_hash_netiface.c b/net/netfilter/ipset/ip_set_hash_netiface.c
index 51e5df12bd00..3d6c53b6211a 100644
--- a/net/netfilter/ipset/ip_set_hash_netiface.c
+++ b/net/netfilter/ipset/ip_set_hash_netiface.c
@@ -100,7 +100,7 @@ iface_test(struct rb_root *root, const char **iface)
         while (n) {
                 const char *d = iface_data(n);
                 int res = ifname_compare(*iface, d);
-                
+
                 if (res < 0)
                         n = n->rb_left;
                 else if (res > 0)
@@ -118,7 +118,7 @@ iface_add(struct rb_root *root, const char **iface)
 {
         struct rb_node **n = &(root->rb_node), *p = NULL;
         struct iface_node *d;
-        
+
         while (*n) {
                 char *ifname = iface_data(*n);
                 int res = ifname_compare(*iface, ifname);
@@ -296,10 +296,10 @@ hash_netiface4_kadt(struct ip_set *set, const struct sk_buff *skb,
         if (opt->cmdflags & IPSET_FLAG_PHYSDEV) {
 #ifdef CONFIG_BRIDGE_NETFILTER
                 const struct nf_bridge_info *nf_bridge = skb->nf_bridge;
-                
+
                 if (!nf_bridge)
                         return -EINVAL;
-                data.iface = SRCDIR ? PHYSDEV(physindev): PHYSDEV(physoutdev);
+                data.iface = SRCDIR ? PHYSDEV(physindev) : PHYSDEV(physoutdev);
                 data.physdev = 1;
 #else
                 data.iface = NULL;
@@ -350,7 +350,7 @@ hash_netiface4_uadt(struct ip_set *set, struct nlattr *tb[],
         if (tb[IPSET_ATTR_CIDR]) {
                 data.cidr = nla_get_u8(tb[IPSET_ATTR_CIDR]);
                 if (!data.cidr)
-                        return -IPSET_ERR_INVALID_CIDR; 
+                        return -IPSET_ERR_INVALID_CIDR;
         }
 
         if (tb[IPSET_ATTR_TIMEOUT]) {
@@ -359,7 +359,7 @@ hash_netiface4_uadt(struct ip_set *set, struct nlattr *tb[],
                 timeout = ip_set_timeout_uget(tb[IPSET_ATTR_TIMEOUT]);
         }
 
-        strcpy(iface, nla_data(tb[IPSET_ATTR_IFACE])); 
+        strcpy(iface, nla_data(tb[IPSET_ATTR_IFACE]));
         data.iface = iface;
         ret = iface_test(&h->rbtree, &data.iface);
         if (adt == IPSET_ADD) {
@@ -372,8 +372,8 @@ hash_netiface4_uadt(struct ip_set *set, struct nlattr *tb[],
                 return ret;
 
         if (tb[IPSET_ATTR_CADT_FLAGS]) {
-                u32 flags = ip_set_get_h32(tb[IPSET_ATTR_CADT_FLAGS]);
-                if (flags & IPSET_FLAG_PHYSDEV)
+                u32 cadt_flags = ip_set_get_h32(tb[IPSET_ATTR_CADT_FLAGS]);
+                if (cadt_flags & IPSET_FLAG_PHYSDEV)
                         data.physdev = 1;
         }
 
@@ -559,10 +559,10 @@ hash_netiface6_kadt(struct ip_set *set, const struct sk_buff *skb,
         if (opt->cmdflags & IPSET_FLAG_PHYSDEV) {
 #ifdef CONFIG_BRIDGE_NETFILTER
                 const struct nf_bridge_info *nf_bridge = skb->nf_bridge;
-                
+
                 if (!nf_bridge)
                         return -EINVAL;
-                data.iface = SRCDIR ? PHYSDEV(physindev): PHYSDEV(physoutdev);
+                data.iface = SRCDIR ? PHYSDEV(physindev) : PHYSDEV(physoutdev);
                 data.physdev = 1;
 #else
                 data.iface = NULL;
@@ -623,7 +623,7 @@ hash_netiface6_uadt(struct ip_set *set, struct nlattr *tb[],
                 timeout = ip_set_timeout_uget(tb[IPSET_ATTR_TIMEOUT]);
         }
 
-        strcpy(iface, nla_data(tb[IPSET_ATTR_IFACE])); 
+        strcpy(iface, nla_data(tb[IPSET_ATTR_IFACE]));
         data.iface = iface;
         ret = iface_test(&h->rbtree, &data.iface);
         if (adt == IPSET_ADD) {
@@ -636,8 +636,8 @@ hash_netiface6_uadt(struct ip_set *set, struct nlattr *tb[],
                 return ret;
 
         if (tb[IPSET_ATTR_CADT_FLAGS]) {
-                u32 flags = ip_set_get_h32(tb[IPSET_ATTR_CADT_FLAGS]);
-                if (flags & IPSET_FLAG_PHYSDEV)
+                u32 cadt_flags = ip_set_get_h32(tb[IPSET_ATTR_CADT_FLAGS]);
+                if (cadt_flags & IPSET_FLAG_PHYSDEV)
                         data.physdev = 1;
         }
 
diff --git a/net/netfilter/ipset/ip_set_hash_netport.c b/net/netfilter/ipset/ip_set_hash_netport.c
index d7710a9fb7c7..fe203d12f56b 100644
--- a/net/netfilter/ipset/ip_set_hash_netport.c
+++ b/net/netfilter/ipset/ip_set_hash_netport.c
@@ -199,7 +199,7 @@ hash_netport4_uadt(struct ip_set *set, struct nlattr *tb[],
         if (tb[IPSET_ATTR_CIDR]) {
                 data.cidr = nla_get_u8(tb[IPSET_ATTR_CIDR]);
                 if (!data.cidr)
-                        return -IPSET_ERR_INVALID_CIDR; 
+                        return -IPSET_ERR_INVALID_CIDR;
         }
 
         if (tb[IPSET_ATTR_PORT])
diff --git a/net/netfilter/ipset/pfxlen.c b/net/netfilter/ipset/pfxlen.c
index b57a85673de7..bd13d66220f1 100644
--- a/net/netfilter/ipset/pfxlen.c
+++ b/net/netfilter/ipset/pfxlen.c
@@ -148,7 +148,7 @@ const union nf_inet_addr ip_set_netmask_map[] = {
 EXPORT_SYMBOL_GPL(ip_set_netmask_map);
 
 #undef  E
-#define E(a, b, c, d)                                           \
+#define E(a, b, c, d)                                           \
         {.ip6 = { (__force __be32) a, (__force __be32) b,       \
                   (__force __be32) c, (__force __be32) d,       \
         } }
diff --git a/net/netfilter/xt_set.c b/net/netfilter/xt_set.c
index 453847f293d3..19461c462dbd 100644
--- a/net/netfilter/xt_set.c
+++ b/net/netfilter/xt_set.c
@@ -37,7 +37,7 @@ match_set(ip_set_id_t index, const struct sk_buff *skb,
         return inv;
 }
 
-#define ADT_OPT(n, f, d, fs, cfs, t)    \
+#define ADT_OPT(n, f, d, fs, cfs, t)    \
 const struct ip_set_adt_opt n = {       \
         .family = f,                    \
         .dim = d,                       \