evaluate: don't crash on set definition with incorrect datatype

Cache updates have resurrected the bug described in 5afa5a164ff1
("evaluate: check for NULL datatype in rhs in lookup expr").

This is triggered by testcases/cache/0008_delete_by_handle_0.

Fixes: df48e56e987f ("cache: add hashtable cache for sets")
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>

diff --git a/src/evaluate.c b/src/evaluate.c
index 95189180fe89bf611e0e07761bbbe4bd4c278f47..2e31ed10ccb798c2aee4144f53b6696f8ab563c0 100644 (file)
--- a/src/evaluate.c
+++ b/src/evaluate.c
@@ -262,7 +262,7 @@ static int expr_evaluate_symbol(struct eval_ctx *ctx, struct expr **expr)
                        return table_not_found(ctx);
 
                set = set_cache_find(table, (*expr)->identifier);
-               if (set == NULL)
+               if (set == NULL || !set->key)
                        return set_not_found(ctx, &(*expr)->location,
                                             (*expr)->identifier);