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/counter.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/counter.c')
| -rw-r--r-- | src/expr/counter.c | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/src/expr/counter.c b/src/expr/counter.c index a1f6574..89a602e 100644 --- a/src/expr/counter.c +++ b/src/expr/counter.c @@ -139,26 +139,10 @@ static int nftnl_expr_counter_snprintf(char *buf, size_t len, uint32_t type, return -1; } -static bool nftnl_expr_counter_cmp(const struct nftnl_expr *e1, - const struct nftnl_expr *e2) -{ - struct nftnl_expr_counter *c1 = nftnl_expr_data(e1); - struct nftnl_expr_counter *c2 = nftnl_expr_data(e2); - bool eq = true; - - if (e1->flags & (1 << NFTNL_EXPR_CTR_PACKETS)) - eq &= (c1->pkts == c2->pkts); - if (e1->flags & (1 << NFTNL_EXPR_CTR_BYTES)) - eq &= (c1->pkts == c2->pkts); - - return eq; -} - struct expr_ops expr_ops_counter = { .name = "counter", .alloc_len = sizeof(struct nftnl_expr_counter), .max_attr = NFTA_COUNTER_MAX, - .cmp = nftnl_expr_counter_cmp, .set = nftnl_expr_counter_set, .get = nftnl_expr_counter_get, .parse = nftnl_expr_counter_parse, |