summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authoranlan_cs <anlan_cs@126.com>2025-01-09 09:36:12 +0100
committeranlan_cs <anlan_cs@126.com>2025-01-10 01:51:40 +0100
commita5ec72aa8cece4f33c24040ff7c527ba1f0baeb7 (patch)
tree11c9f3aeffbc876dda39fcb30d6b43daa0ca4f97
parentMerge pull request #17795 from gromit1811/bugfix_priv_syscaps_alloc (diff)
downloadfrr-a5ec72aa8cece4f33c24040ff7c527ba1f0baeb7.tar.xz
frr-a5ec72aa8cece4f33c24040ff7c527ba1f0baeb7.zip
ospfd: avoid the redundant timers
Since the timer thread for ```OSPF_ROUTE_AGGR_DEL``` has been created, the subsequent "no summary-address" commands shouldn't trigger redundant timers. Signed-off-by: anlan_cs <anlan_cs@126.com>
-rw-r--r--ospfd/ospf_asbr.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/ospfd/ospf_asbr.c b/ospfd/ospf_asbr.c
index aa1146702..978a6fcc1 100644
--- a/ospfd/ospf_asbr.c
+++ b/ospfd/ospf_asbr.c
@@ -1145,8 +1145,7 @@ static void ospf_external_aggr_timer(struct ospf *ospf,
aggr->action = operation;
if (ospf->t_external_aggr) {
- if (ospf->aggr_action == OSPF_ROUTE_AGGR_ADD) {
-
+ if (ospf->aggr_action == OSPF_ROUTE_AGGR_ADD || operation != OSPF_ROUTE_AGGR_ADD) {
if (IS_DEBUG_OSPF(lsa, EXTNL_LSA_AGGR))
zlog_debug("%s: Not required to restart timer,set is already added.",
__func__);