From: Donald Sharp <sharpd@nvidia.com>
Date: Mon, 12 Jul 2021 18:23:51 +0000 (-0400)
Subject: bgpd: hash compare functions never receive null values
X-Git-Tag: base_8.1~331^2
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=refs%2Fpull%2F9036%2Fhead;p=mirror%2Ffrr.git

bgpd: hash compare functions never receive null values

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>
---

diff --git a/bgpd/bgp_evpn.c b/bgpd/bgp_evpn.c
index c99f539c7b..448c0009c5 100644
--- a/bgpd/bgp_evpn.c
+++ b/bgpd/bgp_evpn.c
@@ -145,12 +145,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);
 }
 
@@ -261,12 +255,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);
 }