diff options
| author | Mark Stapp <mjs@cisco.com> | 2025-04-16 11:39:26 -0400 |
|---|---|---|
| committer | Mark Stapp <mjs@cisco.com> | 2025-04-16 13:50:21 -0400 |
| commit | 2d423186259246cba7ba0aaceab2f70a4bce00f0 (patch) | |
| tree | 47d7543d64014696e0c6182ce317cd5999cc79c9 | |
| parent | d378275106e23a72fa9fa7e53617a918977d9b04 (diff) | |
bfdd, bgpd: clean up clang warnings
Clean up some clang compiler warnings.
Signed-off-by: Mark Stapp <mjs@cisco.com>
| -rw-r--r-- | bfdd/bfd.c | 2 | ||||
| -rw-r--r-- | bgpd/bgp_bmp.c | 9 |
2 files changed, 5 insertions, 6 deletions
diff --git a/bfdd/bfd.c b/bfdd/bfd.c index e538aa64c2..f199970e20 100644 --- a/bfdd/bfd.c +++ b/bfdd/bfd.c @@ -2518,7 +2518,7 @@ void sbfd_reflector_free(const uint32_t discr) return; } -void sbfd_reflector_flush() +void sbfd_reflector_flush(void) { sbfd_discr_iterate(_sbfd_reflector_free, NULL); return; diff --git a/bgpd/bgp_bmp.c b/bgpd/bgp_bmp.c index b1bff82b05..e09e7b4941 100644 --- a/bgpd/bgp_bmp.c +++ b/bgpd/bgp_bmp.c @@ -3542,7 +3542,6 @@ static int bmp_bgp_attribute_updated(struct bgp *bgp, bool withdraw) struct bmp_targets *bt; struct listnode *node; struct bmp_imported_bgp *bib; - int ret = 0; struct stream *s = bmp_peerstate(bgp->peer_self, withdraw); struct bmp *bmp; afi_t afi; @@ -3553,8 +3552,8 @@ static int bmp_bgp_attribute_updated(struct bgp *bgp, bool withdraw) if (bmpbgp) { frr_each (bmp_targets, &bmpbgp->targets, bt) { - ret = bmp_bgp_attribute_updated_instance(bt, &bmpbgp->vrf_state, bgp, - withdraw, s); + bmp_bgp_attribute_updated_instance(bt, &bmpbgp->vrf_state, bgp, + withdraw, s); if (withdraw) continue; frr_each (bmp_session, &bt->sessions, bmp) { @@ -3575,8 +3574,8 @@ static int bmp_bgp_attribute_updated(struct bgp *bgp, bool withdraw) frr_each (bmp_imported_bgps, &bt->imported_bgps, bib) { if (bgp_lookup_by_name(bib->name) != bgp) continue; - ret += bmp_bgp_attribute_updated_instance(bt, &bib->vrf_state, bgp, - withdraw, s); + bmp_bgp_attribute_updated_instance(bt, &bib->vrf_state, bgp, + withdraw, s); if (withdraw) continue; frr_each (bmp_session, &bt->sessions, bmp) { |