From b19d239323dd9f732b24ad6c95101cf7a51c4b20 Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Tue, 17 Sep 2019 17:53:31 +0200 Subject: xtables_error() does not return It's a define which resolves into a callback which in turn is declared with noreturn attribute. It will never return, therefore drop all explicit exit() calls or other dead code immediately following it. Signed-off-by: Phil Sutter Acked-by: Florian Westphal --- iptables/xtables-restore.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'iptables/xtables-restore.c') diff --git a/iptables/xtables-restore.c b/iptables/xtables-restore.c index f930f5ba..27e65b97 100644 --- a/iptables/xtables-restore.c +++ b/iptables/xtables-restore.c @@ -131,12 +131,11 @@ void xtables_restore_parse(struct nft_handle *h, table = strtok(buffer+1, " \t\n"); DEBUGP("line %u, table '%s'\n", line, table); - if (!table) { + if (!table) xtables_error(PARAMETER_PROBLEM, "%s: line %u table name invalid\n", xt_params->program_name, line); - exit(1); - } + curtable = nft_table_builtin_find(h, table); if (!curtable) xtables_error(PARAMETER_PROBLEM, @@ -168,12 +167,10 @@ void xtables_restore_parse(struct nft_handle *h, chain = strtok(buffer+1, " \t\n"); DEBUGP("line %u, chain '%s'\n", line, chain); - if (!chain) { + if (!chain) xtables_error(PARAMETER_PROBLEM, "%s: line %u chain name invalid\n", xt_params->program_name, line); - exit(1); - } if (strlen(chain) >= XT_EXTENSION_MAXNAMELEN) xtables_error(PARAMETER_PROBLEM, @@ -183,12 +180,10 @@ void xtables_restore_parse(struct nft_handle *h, policy = strtok(NULL, " \t\n"); DEBUGP("line %u, policy '%s'\n", line, policy); - if (!policy) { + if (!policy) xtables_error(PARAMETER_PROBLEM, "%s: line %u policy invalid\n", xt_params->program_name, line); - exit(1); - } if (nft_chain_builtin_find(curtable, chain)) { if (counters) { -- cgit v1.2.3