netfilter: ipset: Return bool values instead of int
Signed-off-by: Sergey Popovich <popovich_sergei@mail.ua> Signed-off-by: Jozsef Kadlecsik <kadlec@blackhole.kfki.hu> Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
This commit is contained in:
parent
cabfd139aa
commit
728a7e6903
|
@ -56,15 +56,15 @@ hash_ip4_data_equal(const struct hash_ip4_elem *e1,
|
||||||
return e1->ip == e2->ip;
|
return e1->ip == e2->ip;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline bool
|
static bool
|
||||||
hash_ip4_data_list(struct sk_buff *skb, const struct hash_ip4_elem *e)
|
hash_ip4_data_list(struct sk_buff *skb, const struct hash_ip4_elem *e)
|
||||||
{
|
{
|
||||||
if (nla_put_ipaddr4(skb, IPSET_ATTR_IP, e->ip))
|
if (nla_put_ipaddr4(skb, IPSET_ATTR_IP, e->ip))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
@ -198,10 +198,10 @@ hash_ip6_data_list(struct sk_buff *skb, const struct hash_ip6_elem *e)
|
||||||
{
|
{
|
||||||
if (nla_put_ipaddr6(skb, IPSET_ATTR_IP, &e->ip.in6))
|
if (nla_put_ipaddr6(skb, IPSET_ATTR_IP, &e->ip.in6))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
|
@ -63,10 +63,10 @@ hash_ipmark4_data_list(struct sk_buff *skb,
|
||||||
if (nla_put_ipaddr4(skb, IPSET_ATTR_IP, data->ip) ||
|
if (nla_put_ipaddr4(skb, IPSET_ATTR_IP, data->ip) ||
|
||||||
nla_put_net32(skb, IPSET_ATTR_MARK, htonl(data->mark)))
|
nla_put_net32(skb, IPSET_ATTR_MARK, htonl(data->mark)))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
@ -193,10 +193,10 @@ hash_ipmark6_data_list(struct sk_buff *skb,
|
||||||
if (nla_put_ipaddr6(skb, IPSET_ATTR_IP, &data->ip.in6) ||
|
if (nla_put_ipaddr6(skb, IPSET_ATTR_IP, &data->ip.in6) ||
|
||||||
nla_put_net32(skb, IPSET_ATTR_MARK, htonl(data->mark)))
|
nla_put_net32(skb, IPSET_ATTR_MARK, htonl(data->mark)))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
|
@ -69,10 +69,10 @@ hash_ipport4_data_list(struct sk_buff *skb,
|
||||||
nla_put_net16(skb, IPSET_ATTR_PORT, data->port) ||
|
nla_put_net16(skb, IPSET_ATTR_PORT, data->port) ||
|
||||||
nla_put_u8(skb, IPSET_ATTR_PROTO, data->proto))
|
nla_put_u8(skb, IPSET_ATTR_PROTO, data->proto))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
@ -230,10 +230,10 @@ hash_ipport6_data_list(struct sk_buff *skb,
|
||||||
nla_put_net16(skb, IPSET_ATTR_PORT, data->port) ||
|
nla_put_net16(skb, IPSET_ATTR_PORT, data->port) ||
|
||||||
nla_put_u8(skb, IPSET_ATTR_PROTO, data->proto))
|
nla_put_u8(skb, IPSET_ATTR_PROTO, data->proto))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
|
@ -70,10 +70,10 @@ hash_ipportip4_data_list(struct sk_buff *skb,
|
||||||
nla_put_net16(skb, IPSET_ATTR_PORT, data->port) ||
|
nla_put_net16(skb, IPSET_ATTR_PORT, data->port) ||
|
||||||
nla_put_u8(skb, IPSET_ATTR_PROTO, data->proto))
|
nla_put_u8(skb, IPSET_ATTR_PROTO, data->proto))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
@ -239,10 +239,10 @@ hash_ipportip6_data_list(struct sk_buff *skb,
|
||||||
nla_put_net16(skb, IPSET_ATTR_PORT, data->port) ||
|
nla_put_net16(skb, IPSET_ATTR_PORT, data->port) ||
|
||||||
nla_put_u8(skb, IPSET_ATTR_PROTO, data->proto))
|
nla_put_u8(skb, IPSET_ATTR_PROTO, data->proto))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
|
@ -114,10 +114,10 @@ hash_ipportnet4_data_list(struct sk_buff *skb,
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
@ -366,10 +366,10 @@ hash_ipportnet6_data_list(struct sk_buff *skb,
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
|
@ -52,7 +52,12 @@ hash_mac4_data_equal(const struct hash_mac4_elem *e1,
|
||||||
static inline bool
|
static inline bool
|
||||||
hash_mac4_data_list(struct sk_buff *skb, const struct hash_mac4_elem *e)
|
hash_mac4_data_list(struct sk_buff *skb, const struct hash_mac4_elem *e)
|
||||||
{
|
{
|
||||||
return nla_put(skb, IPSET_ATTR_ETHER, ETH_ALEN, e->ether);
|
if (nla_put(skb, IPSET_ATTR_ETHER, ETH_ALEN, e->ether))
|
||||||
|
goto nla_put_failure;
|
||||||
|
return false;
|
||||||
|
|
||||||
|
nla_put_failure:
|
||||||
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
|
@ -95,10 +95,10 @@ hash_net4_data_list(struct sk_buff *skb, const struct hash_net4_elem *data)
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
@ -266,10 +266,10 @@ hash_net6_data_list(struct sk_buff *skb, const struct hash_net6_elem *data)
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
|
@ -193,10 +193,10 @@ hash_netiface4_data_list(struct sk_buff *skb,
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
@ -446,10 +446,10 @@ hash_netiface6_data_list(struct sk_buff *skb,
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
|
@ -110,10 +110,10 @@ hash_netport4_data_list(struct sk_buff *skb,
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
@ -325,10 +325,10 @@ hash_netport6_data_list(struct sk_buff *skb,
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
|
@ -124,10 +124,10 @@ hash_netportnet4_data_list(struct sk_buff *skb,
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
@ -399,10 +399,10 @@ hash_netportnet6_data_list(struct sk_buff *skb,
|
||||||
(flags &&
|
(flags &&
|
||||||
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
nla_put_net32(skb, IPSET_ATTR_CADT_FLAGS, htonl(flags))))
|
||||||
goto nla_put_failure;
|
goto nla_put_failure;
|
||||||
return 0;
|
return false;
|
||||||
|
|
||||||
nla_put_failure:
|
nla_put_failure:
|
||||||
return 1;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
static inline void
|
static inline void
|
||||||
|
|
Loading…
Reference in New Issue