diff options
| author | Rajesh Girada <rgirada@vmware.com> | 2024-05-20 18:34:41 +0200 |
|---|---|---|
| committer | Rajesh Girada <rgirada@vmware.com> | 2024-06-06 13:34:39 +0200 |
| commit | a41b484f0a12b49ebbe70a6dd64b5dc5fdbf99ce (patch) | |
| tree | 604de0c8acd6b3d420649476bb03514a84aba50b | |
| parent | Merge pull request #16022 from opensourcerouting/fix/match_peer (diff) | |
| download | frr-a41b484f0a12b49ebbe70a6dd64b5dc5fdbf99ce.tar.xz frr-a41b484f0a12b49ebbe70a6dd64b5dc5fdbf99ce.zip | |
ospf6d: Handling Topo Change in GR-HELPER mode for max-age lsas
Description:
OSPF6 GR HELPER router should consider as TOPOCHANGE when
it receives lsas with max age and should exit from Helper.
But, it is not exiting from helper because this max age lsa is
considered as duplicated lsa since the sender uses same seq
number for max age lsa from the previous lsa update.
Currently, topo change is not considered for duplicated lsas.
So removed the duplicated check when validating TOPOCHNAGE.
Signed-off-by: Rajesh Girada <rgirada@vmware.com>
| -rw-r--r-- | ospf6d/ospf6_flood.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/ospf6d/ospf6_flood.c b/ospf6d/ospf6_flood.c index 98d3bbc51..b87aa2ffe 100644 --- a/ospf6d/ospf6_flood.c +++ b/ospf6d/ospf6_flood.c @@ -295,9 +295,7 @@ void ospf6_install_lsa(struct ospf6_lsa *lsa) lsa->installed = now; /* Topo change handling */ - if (CHECK_LSA_TOPO_CHG_ELIGIBLE(ntohs(lsa->header->type)) - && !CHECK_FLAG(lsa->flag, OSPF6_LSA_DUPLICATE)) { - + if (CHECK_LSA_TOPO_CHG_ELIGIBLE(ntohs(lsa->header->type))) { /* check if it is new lsa ? or existing lsa got modified ?*/ if (!old || OSPF6_LSA_IS_CHANGED(old, lsa)) ospf6_helper_handle_topo_chg(ospf6, lsa); |