Do not append a space to data regs, they may appear at end of line or followed by a tab. Have callers print the space if needed. Signed-off-by: Phil Sutter --- src/expr/bitwise.c | 7 +++++-- src/expr/cmp.c | 3 +++ src/expr/data_reg.c | 11 +++++------ src/expr/immediate.c | 3 +++ src/expr/range.c | 6 ++++++ src/set_elem.c | 4 ++-- 6 files changed, 24 insertions(+), 10 deletions(-) diff --git a/src/expr/bitwise.c b/src/expr/bitwise.c index cac47a5500993..2da83b7ba0441 100644 --- a/src/expr/bitwise.c +++ b/src/expr/bitwise.c @@ -225,13 +225,16 @@ nftnl_expr_bitwise_snprintf_mask_xor(char *buf, size_t remain, 0, DATA_VALUE); SNPRINTF_BUFFER_SIZE(ret, remain, offset); - ret = snprintf(buf + offset, remain, ") ^ "); + ret = snprintf(buf + offset, remain, " ) ^ "); SNPRINTF_BUFFER_SIZE(ret, remain, offset); ret = nftnl_data_reg_snprintf(buf + offset, remain, &bitwise->xor, 0, DATA_VALUE); SNPRINTF_BUFFER_SIZE(ret, remain, offset); + ret = snprintf(buf + offset, remain, " "); + SNPRINTF_BUFFER_SIZE(ret, remain, offset); + return offset; } @@ -248,7 +251,7 @@ nftnl_expr_bitwise_snprintf_shift(char *buf, size_t remain, const char *op, 0, DATA_VALUE); SNPRINTF_BUFFER_SIZE(ret, remain, offset); - ret = snprintf(buf + offset, remain, ") "); + ret = snprintf(buf + offset, remain, " ) "); SNPRINTF_BUFFER_SIZE(ret, remain, offset); return offset; diff --git a/src/expr/cmp.c b/src/expr/cmp.c index 2908f56325b45..ec1dc31894771 100644 --- a/src/expr/cmp.c +++ b/src/expr/cmp.c @@ -163,6 +163,9 @@ nftnl_expr_cmp_snprintf(char *buf, size_t remain, 0, DATA_VALUE); SNPRINTF_BUFFER_SIZE(ret, remain, offset); + ret = snprintf(buf + offset, remain, " "); + SNPRINTF_BUFFER_SIZE(ret, remain, offset); + return offset; } diff --git a/src/expr/data_reg.c b/src/expr/data_reg.c index fd5e0d6e749e1..bf4153c072fd0 100644 --- a/src/expr/data_reg.c +++ b/src/expr/data_reg.c @@ -25,15 +25,14 @@ nftnl_data_reg_value_snprintf_default(char *buf, size_t remain, const union nftnl_data_reg *reg, uint32_t flags) { - const char *pfx = flags & DATA_F_NOPFX ? "" : "0x"; + const char *pfx = flags & DATA_F_NOPFX ? "" : "0x", *sep = ""; int offset = 0, ret, i; - - for (i = 0; i < div_round_up(reg->len, sizeof(uint32_t)); i++) { ret = snprintf(buf + offset, remain, - "%s%.8x ", pfx, reg->val[i]); + "%s%s%.8x", sep, pfx, reg->val[i]); SNPRINTF_BUFFER_SIZE(ret, remain, offset); + sep = " "; } return offset; @@ -46,11 +45,11 @@ nftnl_data_reg_verdict_snprintf_def(char *buf, size_t size, { int remain = size, offset = 0, ret = 0; - ret = snprintf(buf, size, "%s ", nftnl_verdict2str(reg->verdict)); + ret = snprintf(buf, size, "%s", nftnl_verdict2str(reg->verdict)); SNPRINTF_BUFFER_SIZE(ret, remain, offset); if (reg->chain != NULL) { - ret = snprintf(buf + offset, remain, "-> %s ", reg->chain); + ret = snprintf(buf + offset, remain, " -> %s", reg->chain); SNPRINTF_BUFFER_SIZE(ret, remain, offset); } diff --git a/src/expr/immediate.c b/src/expr/immediate.c index f0e0a78d6b794..6dffaf9ce4ad9 100644 --- a/src/expr/immediate.c +++ b/src/expr/immediate.c @@ -201,6 +201,9 @@ nftnl_expr_immediate_snprintf(char *buf, size_t remain, SNPRINTF_BUFFER_SIZE(ret, remain, offset); } + ret = snprintf(buf + offset, remain, " "); + SNPRINTF_BUFFER_SIZE(ret, remain, offset); + return offset; } diff --git a/src/expr/range.c b/src/expr/range.c index 50a8ed092e38f..564d14f0edbbb 100644 --- a/src/expr/range.c +++ b/src/expr/range.c @@ -176,10 +176,16 @@ static int nftnl_expr_range_snprintf(char *buf, size_t remain, 0, DATA_VALUE); SNPRINTF_BUFFER_SIZE(ret, remain, offset); + ret = snprintf(buf + offset, remain, " "); + SNPRINTF_BUFFER_SIZE(ret, remain, offset); + ret = nftnl_data_reg_snprintf(buf + offset, remain, &range->data_to, 0, DATA_VALUE); SNPRINTF_BUFFER_SIZE(ret, remain, offset); + ret = snprintf(buf + offset, remain, " "); + SNPRINTF_BUFFER_SIZE(ret, remain, offset); + return offset; } diff --git a/src/set_elem.c b/src/set_elem.c index 6c1be44ce5073..f567a28719d11 100644 --- a/src/set_elem.c +++ b/src/set_elem.c @@ -720,12 +720,12 @@ int nftnl_set_elem_snprintf_default(char *buf, size_t remain, DATA_F_NOPFX, dregtype); SNPRINTF_BUFFER_SIZE(ret, remain, offset); } else if (e->flags & (1 << NFTNL_SET_ELEM_OBJREF)) { - ret = snprintf(buf + offset, remain, " : %s ", e->objref); + ret = snprintf(buf + offset, remain, " : %s", e->objref); SNPRINTF_BUFFER_SIZE(ret, remain, offset); } if (e->set_elem_flags) { - ret = snprintf(buf + offset, remain, "flags %u ", e->set_elem_flags); + ret = snprintf(buf + offset, remain, " flags %u", e->set_elem_flags); SNPRINTF_BUFFER_SIZE(ret, remain, offset); } -- 2.51.0