Due to reduced indenting level, some linebreaks are no longer needed. OTOH, strings should not be split to aid in grepping for error output. Signed-off-by: Phil Sutter <phil@xxxxxx> --- iptables/xtables-restore.c | 28 +++++++++++----------------- 1 file changed, 11 insertions(+), 17 deletions(-) diff --git a/iptables/xtables-restore.c b/iptables/xtables-restore.c index 1a3739b3350c8..e9cdf2093bfcb 100644 --- a/iptables/xtables-restore.c +++ b/iptables/xtables-restore.c @@ -135,8 +135,7 @@ static void xtables_restore_parse_line(struct nft_handle *h, return; if (h->noflush == 0) { - DEBUGP("Cleaning all chains of table '%s'\n", - table); + DEBUGP("Cleaning all chains of table '%s'\n", table); if (cb->table_flush) cb->table_flush(h, table); } @@ -161,8 +160,7 @@ static void xtables_restore_parse_line(struct nft_handle *h, if (strlen(chain) >= XT_EXTENSION_MAXNAMELEN) xtables_error(PARAMETER_PROBLEM, - "Invalid chain name `%s' " - "(%u chars max)", + "Invalid chain name `%s' (%u chars max)", chain, XT_EXTENSION_MAXNAMELEN - 1); policy = strtok(NULL, " \t\n"); @@ -179,16 +177,15 @@ static void xtables_restore_parse_line(struct nft_handle *h, if (!ctrs || !parse_counters(ctrs, &count)) xtables_error(PARAMETER_PROBLEM, - "invalid policy counters " - "for chain '%s'\n", chain); + "invalid policy counters for chain '%s'\n", + chain); } if (cb->chain_set && cb->chain_set(h, p->curtable->name, chain, policy, &count) < 0) { xtables_error(OTHER_PROBLEM, - "Can't set policy `%s'" - " on `%s' line %u: %s\n", + "Can't set policy `%s' on `%s' line %u: %s\n", policy, chain, line, ops->strerror(errno)); } @@ -197,15 +194,13 @@ static void xtables_restore_parse_line(struct nft_handle *h, } else if (cb->chain_restore(h, chain, p->curtable->name) < 0 && errno != EEXIST) { xtables_error(PARAMETER_PROBLEM, - "cannot create chain " - "'%s' (%s)\n", chain, - strerror(errno)); + "cannot create chain '%s' (%s)\n", + chain, strerror(errno)); } else if (h->family == NFPROTO_BRIDGE && !ebt_set_user_chain_policy(h, p->curtable->name, chain, policy)) { xtables_error(OTHER_PROBLEM, - "Can't set policy `%s'" - " on `%s' line %u: %s\n", + "Can't set policy `%s' on `%s' line %u: %s\n", policy, chain, line, ops->strerror(errno)); } @@ -238,8 +233,7 @@ static void xtables_restore_parse_line(struct nft_handle *h, for (a = 0; a < newargc; a++) DEBUGP("argv[%u]: %s\n", a, newargv[a]); - ret = cb->do_command(h, newargc, newargv, - &newargv[2], true); + ret = cb->do_command(h, newargc, newargv, &newargv[2], true); if (ret < 0) { if (cb->abort) ret = cb->abort(h); @@ -247,8 +241,8 @@ static void xtables_restore_parse_line(struct nft_handle *h, ret = 0; if (ret < 0) { - fprintf(stderr, "failed to abort " - "commit operation\n"); + fprintf(stderr, + "failed to abort commit operation\n"); } exit(1); } -- 2.23.0