summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorPablo Neira Ayuso <pablo@netfilter.org>2017-02-25 12:01:42 +0100
committerPablo Neira Ayuso <pablo@netfilter.org>2017-02-25 13:40:34 +0100
commitf2406b0014198e821a6fe0e663bec33e5db77f99 (patch)
tree61a750bc877eff78c5233fcc4e62a79a1a9e1810 /src
parent10bdec0add3dcb7c0115575c24a440c66c24f6fd (diff)
netlink_delinearize: remove integer_type_postprocess()
Not required anymore since the set definition now comes with the right byteorder for integer types via NFTA_SET_USERDATA area. So we don't need to look at the lhs anymore. Note that this was a workaround that does not work with named sets, where we cannot assume we have a lhs, since it is valid to have a named set that is not referenced from any rule. Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src')
-rw-r--r--src/netlink_delinearize.c29
1 files changed, 0 insertions, 29 deletions
diff --git a/src/netlink_delinearize.c b/src/netlink_delinearize.c
index 90553836..57b8fa51 100644
--- a/src/netlink_delinearize.c
+++ b/src/netlink_delinearize.c
@@ -1277,33 +1277,6 @@ struct stmt *netlink_parse_set_expr(const struct set *set,
static void expr_postprocess(struct rule_pp_ctx *ctx, struct expr **exprp);
-static void integer_type_postprocess(struct expr *expr)
-{
- struct expr *i;
-
- switch (expr->ops->type) {
- case EXPR_VALUE:
- if (expr->byteorder == BYTEORDER_HOST_ENDIAN) {
- uint32_t len = div_round_up(expr->len, BITS_PER_BYTE);
-
- mpz_switch_byteorder(expr->value, len);
- }
- break;
- case EXPR_SET_REF:
- list_for_each_entry(i, &expr->set->init->expressions, list) {
- expr_set_type(i, expr->dtype, expr->byteorder);
- integer_type_postprocess(i);
- }
- break;
- case EXPR_SET_ELEM:
- expr_set_type(expr->key, expr->dtype, expr->byteorder);
- integer_type_postprocess(expr->key);
- break;
- default:
- break;
- }
-}
-
static void payload_match_expand(struct rule_pp_ctx *ctx,
struct expr *expr,
struct expr *payload)
@@ -1392,8 +1365,6 @@ static void ct_meta_common_postprocess(const struct expr *expr)
break;
case OP_LOOKUP:
expr_set_type(right, left->dtype, left->byteorder);
- if (right->dtype == &integer_type)
- integer_type_postprocess(right);
break;
default: