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/target.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/target.c')
-rw-r--r-- | src/expr/target.c | 20 |
1 files changed, 0 insertions, 20 deletions
diff --git a/src/expr/target.c b/src/expr/target.c index bf2ba77..9100038 100644 --- a/src/expr/target.c +++ b/src/expr/target.c @@ -189,31 +189,11 @@ static void nftnl_expr_target_free(const struct nftnl_expr *e) xfree(target->data); } -static bool nftnl_expr_target_cmp(const struct nftnl_expr *e1, - const struct nftnl_expr *e2) -{ - struct nftnl_expr_target *t1 = nftnl_expr_data(e1); - struct nftnl_expr_target *t2 = nftnl_expr_data(e2); - bool eq = true; - - if (e1->flags & (1 << NFTNL_EXPR_TG_NAME)) - eq &= !strcmp(t1->name, t2->name); - if (e1->flags & (1 << NFTNL_EXPR_TG_REV)) - eq &= (t1->rev == t2->rev); - if (e1->flags & (1 << NFTNL_EXPR_TG_INFO)) { - eq &= (t1->data_len == t2->data_len); - eq &= !memcmp(t1->data, t2->data, t1->data_len); - } - - return eq; -} - struct expr_ops expr_ops_target = { .name = "target", .alloc_len = sizeof(struct nftnl_expr_target), .max_attr = NFTA_TARGET_MAX, .free = nftnl_expr_target_free, - .cmp = nftnl_expr_target_cmp, .set = nftnl_expr_target_set, .get = nftnl_expr_target_get, .parse = nftnl_expr_target_parse, |