]> git.puffer.fish Git - mirror/frr.git/commitdiff
bgpd: Clean up issues found in SA with clang 14
authorDonald Sharp <sharpd@nvidia.com>
Fri, 5 Jan 2024 17:53:34 +0000 (12:53 -0500)
committerDonald Sharp <sharpd@nvidia.com>
Tue, 9 Jan 2024 18:03:43 +0000 (13:03 -0500)
3 different issues:

a) length never used
b) safi never used
c) length never used.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
bgpd/bgp_snmp_bgp4v2.c

index 48db73a343ca36598cf0ed4637da01559826a9f7..0c8ed33d43ae47f35a985d82054beb2d7103612a 100644 (file)
@@ -436,7 +436,6 @@ bgp4v2PathAttrLookup(struct variable *v, oid name[], size_t *length,
        struct bgp_path_info *path, *min;
        struct bgp_dest *dest;
        union sockunion su;
-       unsigned int len;
        struct ipaddr paddr = {};
        size_t namelen = v ? v->namelen : BGP4V2_NLRI_ENTRY_OFFSET;
        sa_family_t family;
@@ -544,11 +543,9 @@ bgp4v2PathAttrLookup(struct variable *v, oid name[], size_t *length,
        /* Set OID offset for prefix type */
        offset = name + namelen;
        offsetlen = *length - namelen;
-       len = offsetlen;
 
        if (offsetlen == 0) {
                dest = bgp_table_top(bgp->rib[afi][SAFI_UNICAST]);
-               safi = SAFI_UNICAST;
        } else {
 
                /* bgp4V2NlriAfi  is already get  */
@@ -595,11 +592,6 @@ bgp4v2PathAttrLookup(struct variable *v, oid name[], size_t *length,
                return NULL;
 
        if (offsetlen > 0) {
-               len = offsetlen;
-               if (len > afi_len)
-                       len = afi_len;
-
-
                /* get bgp4V2PeerRemoteAddrType */
                peer_addr_type = *offset;
                if (peer_addr_type == IANA_AFI_IPV4)