This was overlooked when merging argv-related code: newargc is initialized at declaration and reset in free_argv() again. Fixes: a2ed880a19d08 ("xshared: Consolidate argv construction routines") Signed-off-by: Phil Sutter <phil@xxxxxx> --- iptables/xtables-restore.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/iptables/xtables-restore.c b/iptables/xtables-restore.c index df8844208c273..bb6ee78933f7a 100644 --- a/iptables/xtables-restore.c +++ b/iptables/xtables-restore.c @@ -232,9 +232,6 @@ void xtables_restore_parse(struct nft_handle *h, char *bcnt = NULL; char *parsestart = buffer; - /* reset the newargv */ - newargc = 0; - add_argv(xt_params->program_name, 0); add_argv("-t", 0); add_argv(curtable->name, 0); -- 2.23.0