diff options
author | Pablo Neira Ayuso <pablo@netfilter.org> | 2018-10-24 19:18:50 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2018-10-24 19:26:34 +0200 |
commit | 6dc1c3d8bb64077da7f3f28c7368fb087d10a492 (patch) | |
tree | 880fe8bacaaf4b72035a9c430f86e6e5a15ba9bf /src/expr/masq.c | |
parent | dc240913458d591f59b52b3899d3fc3c5d6ec6ce (diff) |
src: remove nftnl_rule_cmp() and nftnl_expr_cmp()
These functions are not very useful, rule comparison from this layer
does not work well with implicit set definitions.
This infrastructure was added as an attempt to support for deletion by
name from this layer, which should be done from higher level libnftables
library. Finally, we never had a client for this code in
git.netfilter.org.
Let's remove it and bump libversion for safety reasons.
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/expr/masq.c')
-rw-r--r-- | src/expr/masq.c | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/src/expr/masq.c b/src/expr/masq.c index fe017f7..f6f3ceb 100644 --- a/src/expr/masq.c +++ b/src/expr/masq.c @@ -161,28 +161,10 @@ static int nftnl_expr_masq_snprintf(char *buf, size_t len, uint32_t type, return -1; } -static bool nftnl_expr_masq_cmp(const struct nftnl_expr *e1, - const struct nftnl_expr *e2) -{ - struct nftnl_expr_masq *m1 = nftnl_expr_data(e1); - struct nftnl_expr_masq *m2 = nftnl_expr_data(e2); - bool eq = true; - - if (e1->flags & (1 << NFTNL_EXPR_MASQ_FLAGS)) - eq &= (m1->flags == m2->flags); - if (e1->flags & (1 << NFTNL_EXPR_MASQ_REG_PROTO_MIN)) - eq &= (m1->sreg_proto_min == m2->sreg_proto_min); - if (e1->flags & (1 << NFTNL_EXPR_MASQ_REG_PROTO_MAX)) - eq &= (m1->sreg_proto_max == m2->sreg_proto_max); - - return eq; -} - struct expr_ops expr_ops_masq = { .name = "masq", .alloc_len = sizeof(struct nftnl_expr_masq), .max_attr = NFTA_MASQ_MAX, - .cmp = nftnl_expr_masq_cmp, .set = nftnl_expr_masq_set, .get = nftnl_expr_masq_get, .parse = nftnl_expr_masq_parse, |