summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorFlorian Westphal <fw@strlen.de>2019-12-10 15:23:35 +0100
committerFlorian Westphal <fw@strlen.de>2019-12-17 23:10:46 +0100
commitc4e54f81ac1fb9afa9cf78baf9411a6ba2b626a6 (patch)
treeb3b23418e5b6d023ffaf9318e5af2b91e85fa055 /src
parente7069f617fb70dccb52f3de1beab7851a08eade9 (diff)
evaluate: print a hint about 'typeof' syntax on 0 keylen
If user says 'type integer; ...' in a set definition, don't just throw an error -- provide a hint that the typeof keyword can be used to provide the needed size information. Signed-off-by: Florian Westphal <fw@strlen.de>
Diffstat (limited to 'src')
-rw-r--r--src/evaluate.c23
1 files changed, 18 insertions, 5 deletions
diff --git a/src/evaluate.c b/src/evaluate.c
index 91d6b254..817b2322 100644
--- a/src/evaluate.c
+++ b/src/evaluate.c
@@ -3307,6 +3307,20 @@ static int setelem_evaluate(struct eval_ctx *ctx, struct expr **expr)
return 0;
}
+static int set_key_data_error(struct eval_ctx *ctx, const struct set *set,
+ const struct datatype *dtype,
+ const char *name)
+{
+ const char *hint = "";
+
+ if (dtype->size == 0)
+ hint = ". Try \"typeof expression\" instead of \"type datatype\".";
+
+ return set_error(ctx, set, "unqualified type %s "
+ "specified in %s definition%s",
+ dtype->name, name, hint);
+}
+
static int set_evaluate(struct eval_ctx *ctx, struct set *set)
{
struct table *table;
@@ -3331,9 +3345,8 @@ static int set_evaluate(struct eval_ctx *ctx, struct set *set)
return -1;
if (set->key->len == 0)
- return set_error(ctx, set, "unqualified key type %s "
- "specified in %s definition",
- set->key->dtype->name, type);
+ return set_key_data_error(ctx, set,
+ set->key->dtype, type);
}
if (set->flags & NFT_SET_INTERVAL &&
set->key->etype == EXPR_CONCAT)
@@ -3345,8 +3358,8 @@ static int set_evaluate(struct eval_ctx *ctx, struct set *set)
"specify mapping data type");
if (set->data->len == 0 && set->data->dtype->type != TYPE_VERDICT)
- return set_error(ctx, set, "unqualified mapping data "
- "type specified in map definition");
+ return set_key_data_error(ctx, set,
+ set->data->dtype, type);
} else if (set_is_objmap(set->flags)) {
if (set->data) {
assert(set->data->etype == EXPR_VALUE);