From 1c8a6c57b0936ca24fe2dd8455c47d6a2fb5cb7f Mon Sep 17 00:00:00 2001 From: Pablo Neira Ayuso Date: Fri, 19 Apr 2013 01:51:57 +0200 Subject: src: remove trailing \n from all nft_*_snprintf functions The caller should add it in case it needs it. Signed-off-by: Pablo Neira Ayuso --- examples/nft-chain-get.c | 2 +- examples/nft-rule-get.c | 2 +- examples/nft-set-elem-get.c | 2 +- examples/nft-set-get.c | 2 +- examples/nft-table-get.c | 2 +- src/chain.c | 2 +- src/expr/lookup.c | 2 +- src/rule.c | 2 -- src/set.c | 9 ++++++++- src/set_elem.c | 2 -- src/table.c | 2 +- 11 files changed, 16 insertions(+), 13 deletions(-) diff --git a/examples/nft-chain-get.c b/examples/nft-chain-get.c index b32fc0c..b815523 100644 --- a/examples/nft-chain-get.c +++ b/examples/nft-chain-get.c @@ -37,7 +37,7 @@ static int table_cb(const struct nlmsghdr *nlh, void *data) } nft_chain_snprintf(buf, sizeof(buf), t, *type, 0); - printf("%s", buf); + printf("%s\n", buf); err_free: nft_chain_free(t); diff --git a/examples/nft-rule-get.c b/examples/nft-rule-get.c index 2821afe..5bf41f5 100644 --- a/examples/nft-rule-get.c +++ b/examples/nft-rule-get.c @@ -37,7 +37,7 @@ static int table_cb(const struct nlmsghdr *nlh, void *data) } nft_rule_snprintf(buf, sizeof(buf), t, *type, 0); - printf("%s", buf); + printf("%s\n", buf); err_free: nft_rule_free(t); diff --git a/examples/nft-set-elem-get.c b/examples/nft-set-elem-get.c index 3c45ead..34dfca2 100644 --- a/examples/nft-set-elem-get.c +++ b/examples/nft-set-elem-get.c @@ -36,7 +36,7 @@ static int set_cb(const struct nlmsghdr *nlh, void *data) } nft_set_snprintf(buf, sizeof(buf), t, 0, 0); - printf("%s", buf); + printf("%s\n", buf); err_free: nft_set_free(t); diff --git a/examples/nft-set-get.c b/examples/nft-set-get.c index a07a1d7..d4588ba 100644 --- a/examples/nft-set-get.c +++ b/examples/nft-set-get.c @@ -36,7 +36,7 @@ static int set_cb(const struct nlmsghdr *nlh, void *data) } nft_set_snprintf(buf, sizeof(buf), t, 0, 0); - printf("%s", buf); + printf("%s\n", buf); err_free: nft_set_free(t); diff --git a/examples/nft-table-get.c b/examples/nft-table-get.c index 518e176..b113636 100644 --- a/examples/nft-table-get.c +++ b/examples/nft-table-get.c @@ -37,7 +37,7 @@ static int table_cb(const struct nlmsghdr *nlh, void *data) } nft_table_snprintf(buf, sizeof(buf), t, *type, 0); - printf("%s", buf); + printf("%s\n", buf); err_free: nft_table_free(t); diff --git a/src/chain.c b/src/chain.c index c06e552..fb5d218 100644 --- a/src/chain.c +++ b/src/chain.c @@ -471,7 +471,7 @@ static int nft_chain_snprintf_default(char *buf, size_t size, struct nft_chain * { return snprintf(buf, size, "family=%u table=%s chain=%s type=%s " "hook=%u prio=%d policy=%d use=%d " - "packets=%lu bytes=%lu\n", + "packets=%lu bytes=%lu", c->family, c->table, c->name, c->type, c->hooknum, c->prio, c->policy, c->use, c->packets, c->bytes); } diff --git a/src/expr/lookup.c b/src/expr/lookup.c index 003554b..1046615 100644 --- a/src/expr/lookup.c +++ b/src/expr/lookup.c @@ -169,7 +169,7 @@ nft_rule_expr_lookup_snprintf_default(char *buf, size_t size, { int len = size, offset = 0, ret; - ret = snprintf(buf, len, "set=%s sreg=%u dreg=%u\n", + ret = snprintf(buf, len, "set=%s sreg=%u dreg=%u", l->set_name, l->sreg, l->dreg); SNPRINTF_BUFFER_SIZE(ret, size, len, offset); diff --git a/src/rule.c b/src/rule.c index 849cb41..3945676 100644 --- a/src/rule.c +++ b/src/rule.c @@ -494,8 +494,6 @@ static int nft_rule_snprintf_default(char *buf, size_t size, struct nft_rule *r, ret = expr->ops->snprintf(buf+offset, len, type, flags, expr); SNPRINTF_BUFFER_SIZE(ret, size, len, offset); } - ret = snprintf(buf+offset-1, len, "\n"); - SNPRINTF_BUFFER_SIZE(ret, size, len, offset); return ret; } diff --git a/src/set.c b/src/set.c index 9454ace..ef2d11d 100644 --- a/src/set.c +++ b/src/set.c @@ -270,10 +270,17 @@ int nft_set_snprintf(char *buf, size_t size, struct nft_set *s, int len = size, offset = 0; struct nft_set_elem *elem; - ret = snprintf(buf, size, "set=%s table=%s flags=%x\n", + ret = snprintf(buf, size, "set=%s table=%s flags=%x", s->name, s->table, s->set_flags); SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + /* Empty set? Skip printinf of elements */ + if (list_empty(&s->element_list)) + return offset; + + ret = snprintf(buf+offset, size, "\n"); + SNPRINTF_BUFFER_SIZE(ret, size, len, offset); + list_for_each_entry(elem, &s->element_list, head) { ret = snprintf(buf+offset, size, "\t"); SNPRINTF_BUFFER_SIZE(ret, size, len, offset); diff --git a/src/set_elem.c b/src/set_elem.c index 980faed..a2669ad 100644 --- a/src/set_elem.c +++ b/src/set_elem.c @@ -347,8 +347,6 @@ int nft_set_elem_snprintf(char *buf, size_t size, struct nft_set_elem *e, ret = snprintf(buf+offset, len, "%.8x ", e->data.val[i]); SNPRINTF_BUFFER_SIZE(ret, size, len, offset); } - ret = snprintf(buf+offset, size, "\n"); - SNPRINTF_BUFFER_SIZE(ret, size, len, offset); return offset; } diff --git a/src/table.c b/src/table.c index c963e6e..2d72136 100644 --- a/src/table.c +++ b/src/table.c @@ -197,7 +197,7 @@ static int nft_table_snprintf_xml(char *buf, size_t size, struct nft_table *t) static int nft_table_snprintf_default(char *buf, size_t size, struct nft_table *t) { - return snprintf(buf, size, "table=%s family=%u flags=%x\n", + return snprintf(buf, size, "table=%s family=%u flags=%x", t->name, t->family, t->table_flags); } -- cgit v1.2.3