diff options
author | Pablo Neira Ayuso <pablo@netfilter.org> | 2017-05-26 11:49:21 +0100 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2017-05-29 13:00:38 +0200 |
commit | 73ee6f7f4d76c442fb7a3ba3087061b4eef604e3 (patch) | |
tree | 75bdc2db79046d68fd101b0ce48f96899f3bbd15 /src | |
parent | 9330ccf64c4cc17673549bfb6a88acdbd73a8dd5 (diff) |
expression: don't trim off unary expression on delinearization
This transformation introduces an unnecessary asymmetry between the
linearization and delinearization steps that prevent rule deletion by
name to work fine.
Moreover, do not print htonl and ntonl from unary expression, this
syntax is not allowed by the parser.
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src')
-rw-r--r-- | src/expression.c | 3 | ||||
-rw-r--r-- | src/netlink_delinearize.c | 5 |
2 files changed, 1 insertions, 7 deletions
diff --git a/src/expression.c b/src/expression.c index 5defa63f..55dd3911 100644 --- a/src/expression.c +++ b/src/expression.c @@ -460,10 +460,7 @@ const char *expr_op_symbols[] = { static void unary_expr_print(const struct expr *expr) { - if (expr_op_symbols[expr->op] != NULL) - printf("%s(", expr_op_symbols[expr->op]); expr_print(expr->arg); - printf(")"); } static void unary_expr_clone(struct expr *new, const struct expr *expr) diff --git a/src/netlink_delinearize.c b/src/netlink_delinearize.c index 49dc6a60..b355eff4 100644 --- a/src/netlink_delinearize.c +++ b/src/netlink_delinearize.c @@ -1759,10 +1759,7 @@ static void expr_postprocess(struct rule_pp_ctx *ctx, struct expr **exprp) } case EXPR_UNARY: expr_postprocess(ctx, &expr->arg); - expr_set_type(expr->arg, expr->arg->dtype, !expr->arg->byteorder); - - *exprp = expr_get(expr->arg); - expr_free(expr); + expr_set_type(expr, expr->arg->dtype, !expr->arg->byteorder); break; case EXPR_BINOP: expr_postprocess(ctx, &expr->left); |