@@ -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)
@@ -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);
This transformation introduces an unnecessary asymmetry between the linearization and delinearization steps that prevent rule deletion from definition 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> --- src/expression.c | 3 --- src/netlink_delinearize.c | 5 +---- 2 files changed, 1 insertion(+), 7 deletions(-)