matthieu/frr
1
0
Fork 0
forked from Mirror/frr
Code Pull requests Activity

vtysh, zebra: Fix malformed json output for multiple vrfs in command 'show ip route vrf all json'

Browse source 
Command 'show ip route vrf <vrf_name> json' returns a valid json object,
however if instead of <vrf_name> we specify 'all', we get an invalid json
object, like:

{//vrf1 routes}{//vrf2 routes}{vrf3 routes}

After the fix:

{"vrf1":{//vrf1 routes},"vrf2:{//vrf2 routes},"vrf3":{//vrf3 routes}}

Which is a valid json object, that can be parsed effectively using built-in
modules. The rest of the commands remains unaffected and behave the same.

Signed-off-by: Piotr Suchy <psuchy@akamai.com>
This commit is contained in:
Piotr Suchy 2024-03-28 12:55:35 +01:00
parent d5f17cd51e
commit 0e2fc3d67f
1 changed files with 61 additions and 37 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

88
zebra/zebra_vty.c
View file

@ -60,8 +60,8 @@ struct route_show_ctx {
}; };
static int do_show_ip_route(struct vty *vty, const char *vrf_name, afi_t afi, static int do_show_ip_route(struct vty *vty, const char *vrf_name, afi_t afi,
safi_t safi, bool use_fib, bool use_json, safi_t safi, bool use_fib, json_object *vrf_json,
route_tag_t tag, bool use_json, route_tag_t tag,
const struct prefix *longer_prefix_p, const struct prefix *longer_prefix_p,
bool supernets_only, int type, bool supernets_only, int type,
unsigned short ospf_instance_id, uint32_t tableid, unsigned short ospf_instance_id, uint32_t tableid,
@ -148,8 +148,8 @@ DEFPY (show_ip_rpf,
}; };
return do_show_ip_route(vty, VRF_DEFAULT_NAME, ip ? AFI_IP : AFI_IP6, return do_show_ip_route(vty, VRF_DEFAULT_NAME, ip ? AFI_IP : AFI_IP6,
SAFI_MULTICAST, false, uj, 0, NULL, false, 0, 0, SAFI_MULTICAST, false, NULL, uj, 0, NULL, false,
0, false, &ctx); 0, 0, 0, false, &ctx);
} }
DEFPY (show_ip_rpf_addr, DEFPY (show_ip_rpf_addr,
@ -856,14 +856,13 @@ static void vty_show_ip_route_detail_json(struct vty *vty,
vty_json(vty, json); vty_json(vty, json);
} }
static void do_show_route_helper(struct vty *vty, struct zebra_vrf *zvrf, static void
struct route_table *table, afi_t afi, do_show_route_helper(struct vty *vty, struct zebra_vrf *zvrf,
bool use_fib, route_tag_t tag, struct route_table *table, afi_t afi, bool use_fib,
const struct prefix *longer_prefix_p, json_object *vrf_json, route_tag_t tag,
bool supernets_only, int type, const struct prefix *longer_prefix_p, bool supernets_only,
unsigned short ospf_instance_id, bool use_json, int type, unsigned short ospf_instance_id, bool use_json,
uint32_t tableid, bool show_ng, uint32_t tableid, bool show_ng, struct route_show_ctx *ctx)
struct route_show_ctx *ctx)
{ {
struct route_node *rn; struct route_node *rn;
struct route_entry *re; struct route_entry *re;
@ -885,7 +884,7 @@ static void do_show_route_helper(struct vty *vty, struct zebra_vrf *zvrf,
* => display the VRF and table if specific * => display the VRF and table if specific
*/ */
if (use_json) if (use_json && !vrf_json)
json = json_object_new_object(); json = json_object_new_object();
/* Show all routes. */ /* Show all routes. */
@ -960,7 +959,11 @@ static void do_show_route_helper(struct vty *vty, struct zebra_vrf *zvrf,
if (json_prefix) { if (json_prefix) {
prefix2str(&rn->p, buf, sizeof(buf)); prefix2str(&rn->p, buf, sizeof(buf));
if (!vrf_json)
json_object_object_add(json, buf, json_prefix); json_object_object_add(json, buf, json_prefix);
else
json_object_object_add(vrf_json, buf,
json_prefix);
json_prefix = NULL; json_prefix = NULL;
} }
} }
@ -969,13 +972,15 @@ static void do_show_route_helper(struct vty *vty, struct zebra_vrf *zvrf,
* This is an extremely expensive operation at scale * This is an extremely expensive operation at scale
* and non-pretty reduces memory footprint significantly. * and non-pretty reduces memory footprint significantly.
*/ */
if (use_json) if (use_json && !vrf_json) {
vty_json_no_pretty(vty, json); vty_json_no_pretty(vty, json);
json = NULL;
}
} }
static void do_show_ip_route_all(struct vty *vty, struct zebra_vrf *zvrf, static void do_show_ip_route_all(struct vty *vty, struct zebra_vrf *zvrf,
afi_t afi, bool use_fib, bool use_json, afi_t afi, bool use_fib, json_object *vrf_json,
route_tag_t tag, bool use_json, route_tag_t tag,
const struct prefix *longer_prefix_p, const struct prefix *longer_prefix_p,
bool supernets_only, int type, bool supernets_only, int type,
unsigned short ospf_instance_id, bool show_ng, unsigned short ospf_instance_id, bool show_ng,
@ -995,15 +1000,15 @@ static void do_show_ip_route_all(struct vty *vty, struct zebra_vrf *zvrf,
continue; continue;
do_show_ip_route(vty, zvrf_name(zvrf), afi, SAFI_UNICAST, do_show_ip_route(vty, zvrf_name(zvrf), afi, SAFI_UNICAST,
use_fib, use_json, tag, longer_prefix_p, use_fib, vrf_json, use_json, tag,
supernets_only, type, ospf_instance_id, longer_prefix_p, supernets_only, type,
zrt->tableid, show_ng, ctx); ospf_instance_id, zrt->tableid, show_ng, ctx);
} }
} }
static int do_show_ip_route(struct vty *vty, const char *vrf_name, afi_t afi, static int do_show_ip_route(struct vty *vty, const char *vrf_name, afi_t afi,
safi_t safi, bool use_fib, bool use_json, safi_t safi, bool use_fib, json_object *vrf_json,
route_tag_t tag, bool use_json, route_tag_t tag,
const struct prefix *longer_prefix_p, const struct prefix *longer_prefix_p,
bool supernets_only, int type, bool supernets_only, int type,
unsigned short ospf_instance_id, uint32_t tableid, unsigned short ospf_instance_id, uint32_t tableid,
@ -1038,7 +1043,7 @@ static int do_show_ip_route(struct vty *vty, const char *vrf_name, afi_t afi,
return CMD_SUCCESS; return CMD_SUCCESS;
} }
do_show_route_helper(vty, zvrf, table, afi, use_fib, tag, do_show_route_helper(vty, zvrf, table, afi, use_fib, vrf_json, tag,
longer_prefix_p, supernets_only, type, longer_prefix_p, supernets_only, type,
ospf_instance_id, use_json, tableid, show_ng, ctx); ospf_instance_id, use_json, tableid, show_ng, ctx);
@ -1741,6 +1746,7 @@ DEFPY (show_route,
struct route_show_ctx ctx = { struct route_show_ctx ctx = {
.multi = vrf_all || table_all, .multi = vrf_all || table_all,
}; };
json_object *root_json = NULL;
if (!vrf_is_backend_netns()) { if (!vrf_is_backend_netns()) {
if ((vrf_all || vrf_name) && (table || table_all)) { if ((vrf_all || vrf_name) && (table || table_all)) {
@ -1762,24 +1768,42 @@ DEFPY (show_route,
} }
if (vrf_all) { if (vrf_all) {
if (!!json)
root_json = json_object_new_object();
RB_FOREACH (vrf, vrf_name_head, &vrfs_by_name) { RB_FOREACH (vrf, vrf_name_head, &vrfs_by_name) {
json_object *vrf_json = NULL;
if ((zvrf = vrf->info) == NULL if ((zvrf = vrf->info) == NULL
|| (zvrf->table[afi][SAFI_UNICAST] == NULL)) || (zvrf->table[afi][SAFI_UNICAST] == NULL))
continue; continue;
if (!!json)
vrf_json = json_object_new_object();
if (table_all) if (table_all)
do_show_ip_route_all( do_show_ip_route_all(vty, zvrf, afi, !!fib,
vty, zvrf, afi, !!fib, !!json, tag, vrf_json, !!json, tag,
prefix_str ? prefix : NULL, prefix_str ? prefix : NULL,
!!supernets_only, type, !!supernets_only, type,
ospf_instance_id, !!ng, &ctx); ospf_instance_id, !!ng,
&ctx);
else else
do_show_ip_route( do_show_ip_route(vty, zvrf_name(zvrf), afi,
vty, zvrf_name(zvrf), afi, SAFI_UNICAST, SAFI_UNICAST, !!fib, vrf_json,
!!fib, !!json, tag, !!json, tag,
prefix_str ? prefix : NULL, prefix_str ? prefix : NULL,
!!supernets_only, type, !!supernets_only, type,
ospf_instance_id, table, !!ng, &ctx); ospf_instance_id, table, !!ng,
&ctx);
if (!!json)
json_object_object_add(root_json,
zvrf_name(zvrf),
vrf_json);
}
if (!!json) {
vty_json_no_pretty(vty, root_json);
root_json = NULL;
} }
} else { } else {
vrf_id_t vrf_id = VRF_DEFAULT; vrf_id_t vrf_id = VRF_DEFAULT;
@ -1795,13 +1819,13 @@ DEFPY (show_route,
return CMD_SUCCESS; return CMD_SUCCESS;
if (table_all) if (table_all)
do_show_ip_route_all(vty, zvrf, afi, !!fib, !!json, tag, do_show_ip_route_all(vty, zvrf, afi, !!fib, NULL, !!json,
prefix_str ? prefix : NULL, tag, prefix_str ? prefix : NULL,
!!supernets_only, type, !!supernets_only, type,
ospf_instance_id, !!ng, &ctx); ospf_instance_id, !!ng, &ctx);
else else
do_show_ip_route(vty, vrf->name, afi, SAFI_UNICAST, do_show_ip_route(vty, vrf->name, afi, SAFI_UNICAST,
!!fib, !!json, tag, !!fib, NULL, !!json, tag,
prefix_str ? prefix : NULL, prefix_str ? prefix : NULL,
!!supernets_only, type, !!supernets_only, type,
ospf_instance_id, table, !!ng, &ctx); ospf_instance_id, table, !!ng, &ctx);