]> git.puffer.fish Git - matthieu/frr.git/commitdiff
bgpd: hash compare functions never receive null values
authorDonald Sharp <sharpd@nvidia.com>
Mon, 12 Jul 2021 18:23:51 +0000 (14:23 -0400)
committermergify-bot <noreply@mergify.io>
Sat, 24 Jul 2021 16:11:31 +0000 (16:11 +0000)
There is no need to test for null values in the hash compare
function as that we are guaranteed to send in data in
the hash compare functions.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
(cherry picked from commit 63245a641a028dc991de3a5266052e41d876ab20)

bgpd/bgp_evpn.c

index 8c5829905ef7edc546c46d49081f307a5c8d390c..a2f900d6e059b711bb3d178530e100964e47ca78 100644 (file)
@@ -122,12 +122,6 @@ static bool vrf_import_rt_hash_cmp(const void *p1, const void *p2)
        const struct vrf_irt_node *irt1 = p1;
        const struct vrf_irt_node *irt2 = p2;
 
-       if (irt1 == NULL && irt2 == NULL)
-               return true;
-
-       if (irt1 == NULL || irt2 == NULL)
-               return false;
-
        return (memcmp(irt1->rt.val, irt2->rt.val, ECOMMUNITY_SIZE) == 0);
 }
 
@@ -238,12 +232,6 @@ static bool import_rt_hash_cmp(const void *p1, const void *p2)
        const struct irt_node *irt1 = p1;
        const struct irt_node *irt2 = p2;
 
-       if (irt1 == NULL && irt2 == NULL)
-               return true;
-
-       if (irt1 == NULL || irt2 == NULL)
-               return false;
-
        return (memcmp(irt1->rt.val, irt2->rt.val, ECOMMUNITY_SIZE) == 0);
 }