diff options
| author | Acee <aceelindem@gmail.com> | 2024-05-28 16:02:27 +0200 |
|---|---|---|
| committer | Acee <aceelindem@gmail.com> | 2024-05-28 22:46:18 +0200 |
| commit | 772688d2d3c03d8eeeb711c2fe3735c9e0885498 (patch) | |
| tree | caffcb52376b4ebaa7bab347f0532a5e7c5bb5b6 /ospf6d/ospf6_abr.c | |
| parent | Merge pull request #16089 from LabNConsulting/chopps/docfix (diff) | |
| download | frr-772688d2d3c03d8eeeb711c2fe3735c9e0885498.tar.xz frr-772688d2d3c03d8eeeb711c2fe3735c9e0885498.zip | |
ospf6d: OSPFv3 route change comparision fixed for ASBR-only change
When a router route already exists in the area border routers table
as an ABR and it solely changes its ABR or ASBR status, the change
was missed and border route is not updated. This fixes the comparison
for the router_bits in the ospf6_path structure.
This fixes issue https://github.com/FRRouting/frr/issues/16053 although
the actual problem is not the computing router (r2) and not the OSPFv3
redistribution (r3).
Signed-off-by: Acee <aceelindem@gmail.com>
Diffstat (limited to 'ospf6d/ospf6_abr.c')
| -rw-r--r-- | ospf6d/ospf6_abr.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/ospf6d/ospf6_abr.c b/ospf6d/ospf6_abr.c index d3ff759d3..629b9839e 100644 --- a/ospf6d/ospf6_abr.c +++ b/ospf6d/ospf6_abr.c @@ -1357,9 +1357,10 @@ void ospf6_abr_examin_summary(struct ospf6_lsa *lsa, struct ospf6_area *oa) * does not match with the new entry then add the new route */ if (old_entry_updated == false) { - if ((old == NULL) || (old->type != route->type) - || (old->path.type != route->path.type) - || (old->path.cost != route->path.cost)) + if ((old == NULL) || (old->type != route->type) || + (old->path.type != route->path.type) || + (old->path.cost != route->path.cost) || + (old->path.router_bits != route->path.router_bits)) add_route = true; } |