summaryrefslogtreecommitdiffstats
path: root/bgpd/bgp_pbr.c
diff options
context:
space:
mode:
authorPhilippe Guibert <philippe.guibert@6wind.com>2018-06-13 11:56:35 +0200
committerPhilippe Guibert <philippe.guibert@6wind.com>2018-06-28 08:52:15 +0200
commit1a1f4a4c69f5ea36a72a6c824fe7c0f797d41649 (patch)
treeffd86a404a1a438a23cc4250249dfe4a43a3affb /bgpd/bgp_pbr.c
parentbgpd, lib: share flags values for iptable configuration (diff)
downloadfrr-1a1f4a4c69f5ea36a72a6c824fe7c0f797d41649.tar.xz
frr-1a1f4a4c69f5ea36a72a6c824fe7c0f797d41649.zip
bgpd: do not add default route for flowspec for each FS entry
Because the Flowspec entries are parsed first, then injected to Zebra, there are cases where the install feedback from zebra is not received. This leads to unnecessary add route events, whereas one should be enough. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Diffstat (limited to 'bgpd/bgp_pbr.c')
-rw-r--r--bgpd/bgp_pbr.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/bgpd/bgp_pbr.c b/bgpd/bgp_pbr.c
index bf09a999d..779bf2de6 100644
--- a/bgpd/bgp_pbr.c
+++ b/bgpd/bgp_pbr.c
@@ -1393,7 +1393,7 @@ static void bgp_pbr_policyroute_add_to_zebra(struct bgp *bgp,
* it will be suppressed subsequently
*/
/* ip rule add */
- if (!bpa->installed) {
+ if (!bpa->installed && !bpa->install_in_progress) {
bgp_send_pbr_rule_action(bpa, true);
bgp_zebra_announce_default(bgp, nh,
AFI_IP, bpa->table_id, true);