diff options
author | Florian Westphal <fw@strlen.de> | 2023-12-14 17:56:59 +0100 |
---|---|---|
committer | Florian Westphal <fw@strlen.de> | 2023-12-14 21:50:50 +0100 |
commit | 8eeedce89d8bf0ad58da398782c2ca8a91d83a32 (patch) | |
tree | 72245a4c2335da18b5371b2b0fb1a7a0c1b9bc15 /src | |
parent | 0404ff08b3c18052e6689d75fa85275d3cef7e8e (diff) |
evaluate: exthdr: statement arg must be not be a range
Else we get:
BUG: unknown expression type range
nft: src/netlink_linearize.c:909: netlink_gen_expr: Assertion `0' failed.
Signed-off-by: Florian Westphal <fw@strlen.de>
Diffstat (limited to 'src')
-rw-r--r-- | src/evaluate.c | 19 |
1 files changed, 16 insertions, 3 deletions
diff --git a/src/evaluate.c b/src/evaluate.c index 70d80eb4..1c5078d6 100644 --- a/src/evaluate.c +++ b/src/evaluate.c @@ -3024,14 +3024,27 @@ static bool stmt_evaluate_payload_need_csum(const struct expr *payload) static int stmt_evaluate_exthdr(struct eval_ctx *ctx, struct stmt *stmt) { struct expr *exthdr; + int ret; if (__expr_evaluate_exthdr(ctx, &stmt->exthdr.expr) < 0) return -1; exthdr = stmt->exthdr.expr; - return stmt_evaluate_arg(ctx, stmt, exthdr->dtype, exthdr->len, - BYTEORDER_BIG_ENDIAN, - &stmt->exthdr.val); + ret = stmt_evaluate_arg(ctx, stmt, exthdr->dtype, exthdr->len, + BYTEORDER_BIG_ENDIAN, + &stmt->exthdr.val); + if (ret < 0) + return ret; + + switch (stmt->exthdr.val->etype) { + case EXPR_RANGE: + return expr_error(ctx->msgs, stmt->exthdr.val, + "cannot be a range"); + default: + break; + } + + return 0; } static int stmt_evaluate_payload(struct eval_ctx *ctx, struct stmt *stmt) |