diff options
| author | Olivier Dugeon <olivier.dugeon@orange.com> | 2018-02-03 19:30:33 +0100 |
|---|---|---|
| committer | Olivier Dugeon <olivier.dugeon@orange.com> | 2018-02-03 19:30:33 +0100 |
| commit | bcf4475ec3f3946a5b9b25d67077bc4cac504954 (patch) | |
| tree | 802b43e50fcf74414cb906f9977f12e6ef6744e3 /ospfd/ospf_opaque.c | |
| parent | OSPFd: Correct Extended Prefix LSA refresh (diff) | |
| download | frr-bcf4475ec3f3946a5b9b25d67077bc4cac504954.tar.xz frr-bcf4475ec3f3946a5b9b25d67077bc4cac504954.zip | |
OSPFd: Fix ospfd crash during CI
When preforming CI test, CLI command 'no router ospf' followed by a
'router ospf' is performed to clean up the previous configuration.
Ospfd crash when configuring 'netwoark area'.
This is due to opsf_opaque_term() introduce in previous commit that cause this
crash. It remove not only Opaque LSA but also the list through the call to
'list_delete_and_null()' function. Same take place in 'ospf_mpls_te_term()',
'ospf_router_info_term()' and 'ospf_ext_term()' function.
New set of 'ospf_XXX_finish()' has been introduced to solve this issue while
keeping the possiblity to terminate properly the Opaque LSA and remove MPLS
LFIB entries set by Segment Routing.
Signed-off-by: Olivier Dugeon <olivier.dugeon@orange.com>
Diffstat (limited to 'ospfd/ospf_opaque.c')
| -rw-r--r-- | ospfd/ospf_opaque.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/ospfd/ospf_opaque.c b/ospfd/ospf_opaque.c index 216492781..1c586d252 100644 --- a/ospfd/ospf_opaque.c +++ b/ospfd/ospf_opaque.c @@ -122,6 +122,15 @@ void ospf_opaque_term(void) return; } +void ospf_opaque_finish(void) +{ + ospf_router_info_finish(); + + ospf_ext_finish(); + + ospf_sr_finish(); +} + int ospf_opaque_type9_lsa_init(struct ospf_interface *oi) { if (oi->opaque_lsa_self != NULL) |