bgpd: In bgp_update() for mac addrs ensure we are dealing with evpn

The code is just arbitrarily checking to see if there are any
mac addresses associated with a prefix.  This makes no
sense from the perspective that it can only happen as
an evpn route.  Let's not make non-evpn people pay
the price to check this data.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>

diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c
index 58550c70c92941c973ca571195e4c5c91176d7d0..4a36f1a26d334907091c01ceb0492b8cab3c6579 100644 (file)
--- a/bgpd/bgp_route.c
+++ b/bgpd/bgp_route.c
@@ -4889,7 +4889,7 @@ void bgp_update(struct peer *peer, const struct prefix *p, uint32_t addpath_id,
                goto filtered;
        }
 
-       if (bgp_mac_entry_exists(p) || bgp_mac_exist(&attr->rmac)) {
+       if (safi == SAFI_EVPN && (bgp_mac_entry_exists(p) || bgp_mac_exist(&attr->rmac))) {
                peer->stat_pfx_nh_invalid++;
                reason = "self mac;";
                bgp_attr_flush(&new_attr);