diff options
| author | Zahari Doychev <zdoychev@maxlinear.com> | 2023-06-29 21:57:36 +0200 |
|---|---|---|
| committer | Stephen Hemminger <stephen@networkplumber.org> | 2023-07-03 09:02:22 -0700 |
| commit | 19f44c06e5e276a35758bb154fa4eaddf9f3b5b2 (patch) | |
| tree | e2cf849f06e2f8d74f951c9bb89bf29da496b760 | |
| parent | bbfccc11e1c4399f46921365b36807c3791be536 (diff) | |
| download | iproute2-19f44c06e5e276a35758bb154fa4eaddf9f3b5b2.tar.gz | |
f_flower: simplify cfm dump function
The standard print function can be used to print the cfm attributes in
both standard and json use cases. In this way no string buffer is needed
which simplifies the code.
Signed-off-by: Zahari Doychev <zdoychev@maxlinear.com>
Reviewed-by: Petr Machata <me@pmachata.org>
Reviewed-by: Ido Schimmel <idosch@nvidia.com>
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
| -rw-r--r-- | tc/f_flower.c | 20 |
1 files changed, 6 insertions, 14 deletions
diff --git a/tc/f_flower.c b/tc/f_flower.c index 6da5028ad..c71394f75 100644 --- a/tc/f_flower.c +++ b/tc/f_flower.c @@ -2816,9 +2816,6 @@ static void flower_print_arp_op(const char *name, static void flower_print_cfm(struct rtattr *attr) { struct rtattr *tb[TCA_FLOWER_KEY_CFM_OPT_MAX + 1]; - struct rtattr *v; - SPRINT_BUF(out); - size_t sz = 0; if (!attr || !(attr->rta_type & NLA_F_NESTED)) return; @@ -2830,20 +2827,15 @@ static void flower_print_cfm(struct rtattr *attr) print_string(PRINT_FP, NULL, " cfm", NULL); open_json_object("cfm"); - v = tb[TCA_FLOWER_KEY_CFM_MD_LEVEL]; - if (v) { - sz += sprintf(out, " mdl %u", rta_getattr_u8(v)); - print_hhu(PRINT_JSON, "mdl", NULL, rta_getattr_u8(v)); - } + if (tb[TCA_FLOWER_KEY_CFM_MD_LEVEL]) + print_hhu(PRINT_ANY, "mdl", " mdl %u", + rta_getattr_u8(tb[TCA_FLOWER_KEY_CFM_MD_LEVEL])); - v = tb[TCA_FLOWER_KEY_CFM_OPCODE]; - if (v) { - sprintf(out + sz, " op %u", rta_getattr_u8(v)); - print_hhu(PRINT_JSON, "op", NULL, rta_getattr_u8(v)); - } + if (tb[TCA_FLOWER_KEY_CFM_OPCODE]) + print_hhu(PRINT_ANY, "op", " op %u", + rta_getattr_u8(tb[TCA_FLOWER_KEY_CFM_OPCODE])); close_json_object(); - print_string(PRINT_FP, "cfm", "%s", out); } static int flower_print_opt(struct filter_util *qu, FILE *f, |