diff options
author | Jozsef Kadlecsik <kadlec@blackhole.kfki.hu> | 2011-01-20 14:48:23 +0100 |
---|---|---|
committer | Jozsef Kadlecsik <kadlec@blackhole.kfki.hu> | 2011-01-20 14:48:23 +0100 |
commit | c8396bdc040f4b16e6f6e3f8b81b9fb67a499d9c (patch) | |
tree | 0704d9f117fb9fefed1b9ec5e680e765d80ff877 /kernel/ip_set_hash_ipportip.c | |
parent | 13dcc23a375f61d8286072b20067fb31bcc3ad46 (diff) |
Use annotated types and fix sparse warnings
Annotated types are introduced and sparse warnings fixed.
Two warnings remained in ip_set_core.c but those are false ones.
(Patrick McHardy's review)
Diffstat (limited to 'kernel/ip_set_hash_ipportip.c')
-rw-r--r-- | kernel/ip_set_hash_ipportip.c | 23 |
1 files changed, 11 insertions, 12 deletions
diff --git a/kernel/ip_set_hash_ipportip.c b/kernel/ip_set_hash_ipportip.c index f7a9870..734529c 100644 --- a/kernel/ip_set_hash_ipportip.c +++ b/kernel/ip_set_hash_ipportip.c @@ -46,18 +46,18 @@ hash_ipportip_same_set(const struct ip_set *a, const struct ip_set *b); /* Member elements without timeout */ struct hash_ipportip4_elem { - u32 ip; - u32 ip2; - u16 port; + __be32 ip; + __be32 ip2; + __be16 port; u8 proto; u8 padding; }; /* Member elements with timeout support */ struct hash_ipportip4_telem { - u32 ip; - u32 ip2; - u16 port; + __be32 ip; + __be32 ip2; + __be16 port; u8 proto; u8 padding; unsigned long timeout; @@ -200,7 +200,7 @@ hash_ipportip4_uadt(struct ip_set *set, struct nlattr *head, int len, return ret; if (tb[IPSET_ATTR_PORT]) - data.port = ip_set_get_n16(tb[IPSET_ATTR_PORT]); + data.port = nla_get_be16(tb[IPSET_ATTR_PORT]); else return -IPSET_ERR_PROTOCOL; @@ -238,10 +238,9 @@ hash_ipportip4_uadt(struct ip_set *set, struct nlattr *head, int len, ip = ntohl(data.ip); if (tb[IPSET_ATTR_IP_TO]) { - ret = ip_set_get_ipaddr4(tb, IPSET_ATTR_IP_TO, &ip_to); + ret = ip_set_get_hostipaddr4(tb, IPSET_ATTR_IP_TO, &ip_to); if (ret) return ret; - ip_to = ntohl(ip_to); if (ip > ip_to) swap(ip, ip_to); } else if (tb[IPSET_ATTR_CIDR]) { @@ -292,7 +291,7 @@ hash_ipportip_same_set(const struct ip_set *a, const struct ip_set *b) struct hash_ipportip6_elem { union nf_inet_addr ip; union nf_inet_addr ip2; - u16 port; + __be16 port; u8 proto; u8 padding; }; @@ -300,7 +299,7 @@ struct hash_ipportip6_elem { struct hash_ipportip6_telem { union nf_inet_addr ip; union nf_inet_addr ip2; - u16 port; + __be16 port; u8 proto; u8 padding; unsigned long timeout; @@ -432,7 +431,7 @@ hash_ipportip6_uadt(struct ip_set *set, struct nlattr *head, int len, return ret; if (tb[IPSET_ATTR_PORT]) - data.port = ip_set_get_n16(tb[IPSET_ATTR_PORT]); + data.port = nla_get_be16(tb[IPSET_ATTR_PORT]); else return -IPSET_ERR_PROTOCOL; |