diff options
| author | Fengguang Wu | 2013-12-27 11:13:03 +0100 |
|---|---|---|
| committer | Jozsef Kadlecsik | 2014-03-06 09:31:42 +0100 |
| commit | 9562cf28d1b48d0545d7b5dd2995d00b45e1cb53 (patch) | |
| tree | 94b81097af0e0af685c828048b9fe0975e9d551e /net/netfilter/ipset/ip_set_hash_netnet.c | |
| parent | netfilter: ipset: Follow manual page behavior for SET target on list:set (diff) | |
| download | kernel-qcow2-linux-9562cf28d1b48d0545d7b5dd2995d00b45e1cb53.tar.gz kernel-qcow2-linux-9562cf28d1b48d0545d7b5dd2995d00b45e1cb53.tar.xz kernel-qcow2-linux-9562cf28d1b48d0545d7b5dd2995d00b45e1cb53.zip | |
netfilter: ipset: Add hash: fix coccinelle warnings
net/netfilter/ipset/ip_set_hash_netnet.c:115:8-9: WARNING: return of 0/1 in function 'hash_netnet4_data_list' with return type bool
/c/kernel-tests/src/cocci/net/netfilter/ipset/ip_set_hash_netnet.c:338:8-9: WARNING: return of 0/1 in function 'hash_netnet6_data_list' with return type bool
Return statements in functions returning bool should use
true/false instead of 1/0.
Generated by: coccinelle/misc/boolreturn.cocci
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Jozsef Kadlecsik <kadlec@blackhole.kfki.hu>
Diffstat (limited to 'net/netfilter/ipset/ip_set_hash_netnet.c')
| -rw-r--r-- | net/netfilter/ipset/ip_set_hash_netnet.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/net/netfilter/ipset/ip_set_hash_netnet.c b/net/netfilter/ipset/ip_set_hash_netnet.c index 6226803fc490..4e7261df8961 100644 --- a/net/netfilter/ipset/ip_set_hash_netnet.c +++ b/net/netfilter/ipset/ip_set_hash_netnet.c @@ -112,10 +112,10 @@ hash_netnet4_data_list(struct sk_buff *skb, (flags && nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags)))) goto nla_put_failure; - return 0; + return false; nla_put_failure: - return 1; + return true; } static inline void @@ -334,10 +334,10 @@ hash_netnet6_data_list(struct sk_buff *skb, (flags && nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags)))) goto nla_put_failure; - return 0; + return false; nla_put_failure: - return 1; + return true; } static inline void |