diff options
author | Florian Westphal <fw@strlen.de> | 2024-01-11 16:57:28 +0100 |
---|---|---|
committer | Florian Westphal <fw@strlen.de> | 2024-01-11 18:25:55 +0100 |
commit | 666018e71ebb5df376b1b013c1ca859eaed66f1a (patch) | |
tree | e433b7b18d1f6b46a64dfa4bc4a2a6b93162bf46 /src/evaluate.c | |
parent | 52d1346d41c51abf2f77b81d21cb683a6477e817 (diff) |
evaluate: error out when expression has no datatype
add rule ip6 f i rt2 addr . ip6 daddr { dead:: . dead:: }
... will cause a segmentation fault, we assume expr->dtype is always
set.
rt2 support is incomplete, the template is uninitialised.
This could be fixed up, but rt2 (a subset of the deperecated type 0),
like all other routing headers, lacks correct dependency tracking.
Currently such routing headers are always assumed to be segment routing
headers, we would need to add dependency on 'Routing Type' field in the
routing header, similar to icmp type/code.
Signed-off-by: Florian Westphal <fw@strlen.de>
Diffstat (limited to 'src/evaluate.c')
-rw-r--r-- | src/evaluate.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/evaluate.c b/src/evaluate.c index 41524eef..197c82c2 100644 --- a/src/evaluate.c +++ b/src/evaluate.c @@ -1593,6 +1593,11 @@ static int expr_evaluate_concat(struct eval_ctx *ctx, struct expr **expr) "cannot use %s in concatenation", expr_name(i)); + if (!i->dtype) + return expr_error(ctx->msgs, i, + "cannot use %s in concatenation, lacks datatype", + expr_name(i)); + flags &= i->flags; if (!key && i->dtype->type == TYPE_INTEGER) { |