]> git.puffer.fish Git - mirror/frr.git/commitdiff
bgpd: fix routemap update with disabled delay timer 8828/head
authorIgor Ryzhov <iryzhov@nfware.com>
Thu, 10 Jun 2021 17:21:51 +0000 (20:21 +0300)
committerIgor Ryzhov <iryzhov@nfware.com>
Thu, 10 Jun 2021 17:21:51 +0000 (20:21 +0300)
- vnc_routemap_update is called only for the last bgp instance
- vpn_policy_routemap_event is not called at all
- unguarded debug (there's already a debug inside the called function)

Make the code consistent with the callback code that is used when the
delay timer is enabled.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
bgpd/bgp_routemap.c

index 921a3e0dd9a0306ec63ebc2eb8854e5a4160b280..806a771cfbeba59e0774952b7627b573a03a530a 100644 (file)
@@ -3767,7 +3767,6 @@ static void bgp_route_map_process_update_cb(char *rmap_name)
                bgp_route_map_process_update(bgp, rmap_name, 1);
 
 #ifdef ENABLE_BGP_VNC
-               /* zlog_debug("%s: calling vnc_routemap_update", __func__); */
                vnc_routemap_update(bgp, __func__);
 #endif
        }
@@ -3807,12 +3806,14 @@ static void bgp_route_map_mark_update(const char *rmap_name)
                                                   BGP_POLICY_ROUTE_MAP,
                                                   rmap_name, 1, 1);
        } else {
-               for (ALL_LIST_ELEMENTS(bm->bgp, node, nnode, bgp))
+               for (ALL_LIST_ELEMENTS(bm->bgp, node, nnode, bgp)) {
                        bgp_route_map_process_update(bgp, rmap_name, 0);
 #ifdef ENABLE_BGP_VNC
-               zlog_debug("%s: calling vnc_routemap_update", __func__);
-               vnc_routemap_update(bgp, __func__);
+                       vnc_routemap_update(bgp, __func__);
 #endif
+               }
+
+               vpn_policy_routemap_event(rmap_name);
        }
 }