diff options
author | Florian Westphal <fw@strlen.de> | 2019-12-10 15:23:35 +0100 |
---|---|---|
committer | Florian Westphal <fw@strlen.de> | 2019-12-17 23:10:46 +0100 |
commit | c4e54f81ac1fb9afa9cf78baf9411a6ba2b626a6 (patch) | |
tree | b3b23418e5b6d023ffaf9318e5af2b91e85fa055 /src/evaluate.c | |
parent | e7069f617fb70dccb52f3de1beab7851a08eade9 (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/evaluate.c')
-rw-r--r-- | src/evaluate.c | 23 |
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); |