]> 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:16 +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 1172514e52615dcf3a0846a408b8ae990d93e9de..87f2e55b3f84b18c51cb291515d0c50804b1fa82 100644 (file)
@@ -2597,8 +2597,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: