diff options
| author | Donald Sharp <sharpd@nvidia.com> | 2024-08-13 08:49:01 -0400 |
|---|---|---|
| committer | Donald Sharp <sharpd@nvidia.com> | 2024-08-13 08:49:01 -0400 |
| commit | ce4e451c5b86d3f79c88283e2a7425ce32e9df1b (patch) | |
| tree | ae5b2e8e4c693d32dd8ff0870bba134d94443064 | |
| parent | c4fdc837c0f14292321845f6c61285984bf93bf2 (diff) | |
bgpd: Free up non-freed json memory on function return
json_peers is allocated in the above if statement block
for json but is not freed in this code path. Noticed
by running Address Sanitizer.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
| -rw-r--r-- | bgpd/bgp_vty.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/bgpd/bgp_vty.c b/bgpd/bgp_vty.c index a3180fd707..43e267c5e8 100644 --- a/bgpd/bgp_vty.c +++ b/bgpd/bgp_vty.c @@ -12046,6 +12046,8 @@ static int bgp_show_summary(struct vty *vty, struct bgp *bgp, int afi, int safi, if (show_failed && !failed_count) { if (use_json) { + json_object_free(json_peers); + json_object_int_add(json, "failedPeersCount", 0); json_object_int_add(json, "dynamicPeers", dn_count); json_object_int_add(json, "totalPeers", count); |