]> git.puffer.fish Git - matthieu/frr.git/commitdiff
bgpd: Note when receiving but not understanding a route notification
authorDonald Sharp <sharpd@nvidia.com>
Thu, 28 Mar 2024 16:25:05 +0000 (12:25 -0400)
committerMergify <37929162+mergify[bot]@users.noreply.github.com>
Wed, 3 Apr 2024 06:52:25 +0000 (06:52 +0000)
When BGP has been asked to wait for FIB installation, on route
removal a return call is likely to not have the dest since BGP
will have cleaned up the node, entirely.  Let's just note that
the prefix cannot be found if debugs are turned on and move on.

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

bgpd/bgp_zebra.c

index 18162a7ae093971ce8cc3ee642dbaf6cef62b1f2..34198c96e1eb4748464eaa49aa83a3ef727fa442 100644 (file)
@@ -2619,8 +2619,12 @@ static int bgp_zebra_route_notify_owner(int command, struct zclient *zclient,
        /* Find the bgp route node */
        dest = bgp_safi_node_lookup(bgp->rib[afi][safi], safi, &p,
                                    &bgp->vrf_prd);
-       if (!dest)
+       if (!dest) {
+               if (BGP_DEBUG(zebra, ZEBRA))
+                       zlog_debug("%s: %pFX does not exist in the BGP table, nothing to do for %u",
+                                  __func__, &p, note);
                return -1;
+       }
 
        switch (note) {
        case ZAPI_ROUTE_INSTALLED: