Fix potential null pointer dereference in `do_list_flowtable`. The pointer `table` is initialized to NULL and passed to `do_list_flowtable`, where it may be dereferenced. This can lead to a crash if `table` remains NULL. Changes: - Added a NULL check for the `table` pointer before calling `do_list_flowtable`. - Return an error code (-1) if `table` is NULL to handle the case where the table is not found. Triggers found by static analyzer Svace. Signed-off-by: Anton Moryakov <ant.v.moryakov@xxxxxxxxx> --- src/rule.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/rule.c b/src/rule.c index f7582914..59d3f3ac 100644 --- a/src/rule.c +++ b/src/rule.c @@ -1556,7 +1556,7 @@ static int do_delete_setelems(struct netlink_ctx *ctx, struct cmd *cmd) const struct set *set = cmd->elem.set; struct expr *expr = cmd->elem.expr; - if (set_is_non_concat_range(set) && + if (set && set_is_non_concat_range(set) && set_to_intervals(set, expr, false) < 0) return -1; -- 2.30.2
