From b1392e5d0857a9c2034aad56eafc1ad09b8f2003 Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Fri, 13 Jan 2023 17:09:53 +0100 Subject: [PATCH] optimize: Do not return garbage from stack commit d4d47e5bdf943be494aeb5d5a29b8f5212acbddf upstream. If input does not contain a single 'add' command (unusual, but possible), 'ret' value was not initialized by nft_optimize() before returning its value. Fixes: fb298877ece27 ("src: add ruleset optimization infrastructure") Signed-off-by: Phil Sutter --- src/optimize.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/optimize.c b/src/optimize.c index 13aa1acc..d592aee0 100644 --- a/src/optimize.c +++ b/src/optimize.c @@ -1215,7 +1215,7 @@ static int cmd_optimize(struct nft_ctx *nft, struct cmd *cmd) int nft_optimize(struct nft_ctx *nft, struct list_head *cmds) { struct cmd *cmd; - int ret; + int ret = 0; list_for_each_entry(cmd, cmds, list) { switch (cmd->op) { -- 2.47.3