ospf6d: Disable and delete OSPFv3 areas that no longer have config.

This fix will delete an OSPFv3 area when all the interfaces and
configuration (ranges, NSSA ranges, stub area, NSSA area, filter-list,
import-list and export-list) have been removed. The changes provides
a general solution to https://github.com/FRRouting/frr/issues/18324.

Signed-off-by: Acee Lindem <acee@lindem.com>

diff --git a/ospf6d/ospf6_area.c b/ospf6d/ospf6_area.c
index cf5479e18c8744806640125beb0bd2562061ebb4..a574b0b7ac2475895c077b6bda5394246c2f5d7a 100644 (file)
--- a/ospf6d/ospf6_area.c
+++ b/ospf6d/ospf6_area.c
@@ -366,6 +366,17 @@ void ospf6_area_delete(struct ospf6_area *oa)
        XFREE(MTYPE_OSPF6_AREA, oa);
 }
 
+void ospf6_area_no_config_delete(struct ospf6_area *oa)
+{
+       if ((oa->if_list->count == 0) && (oa->range_table->count == 0) &&
+           (oa->nssa_range_table->count == 0) && !IS_AREA_STUB(oa) && !IS_AREA_NSSA(oa) &&
+           !PREFIX_NAME_IN(oa) && !PREFIX_NAME_OUT(oa) && !IMPORT_NAME(oa) && !EXPORT_NAME(oa)) {
+               if (CHECK_FLAG((oa)->flag, OSPF6_AREA_ENABLE))
+                       ospf6_area_disable(oa);
+               ospf6_area_delete(oa);
+       }
+}
+
 struct ospf6_area *ospf6_area_lookup_by_area_id(uint32_t area_id)
 {
        struct ospf6_area *oa;
@@ -646,6 +657,9 @@ DEFUN (no_area_range,
        }
        ospf6_route_remove(range, oa->range_table);
 
+       /* Delete area if no interfaces or configuration. */
+       ospf6_area_no_config_delete(oa);
+
        return CMD_SUCCESS;
 }
 
@@ -814,6 +828,9 @@ DEFUN (no_area_filter_list,
        if (ospf6_check_and_set_router_abr(area->ospf6))
                ospf6_schedule_abr_task(ospf6);
 
+       /* Delete area if no interfaces or configuration. */
+       ospf6_area_no_config_delete(area);
+
        return CMD_SUCCESS;
 }
 
@@ -939,6 +956,9 @@ DEFUN (no_area_import_list,
        if (ospf6_check_and_set_router_abr(area->ospf6))
                ospf6_schedule_abr_task(ospf6);
 
+       /* Delete area if no interfaces or configuration. */
+       ospf6_area_no_config_delete(area);
+
        return CMD_SUCCESS;
 }
 
@@ -1002,6 +1022,9 @@ DEFUN (no_area_export_list,
        if (ospf6_check_and_set_router_abr(area->ospf6))
                ospf6_schedule_abr_task(ospf6);
 
+       /* Delete area if no interfaces or configuration. */
+       ospf6_area_no_config_delete(area);
+
        return CMD_SUCCESS;
 }
 
@@ -1338,6 +1361,9 @@ DEFUN (no_ospf6_area_stub_no_summary,
        ospf6_area_stub_unset(ospf6, area);
        ospf6_area_no_summary_unset(ospf6, area);
 
+       /* Delete area if no interfaces or configuration. */
+       ospf6_area_no_config_delete(area);
+
        return CMD_SUCCESS;
 }
 
@@ -1418,6 +1444,9 @@ DEFPY(no_ospf6_area_nssa, no_ospf6_area_nssa_cmd,
        ospf6_area_no_summary_unset(ospf6, area);
        ospf6_nssa_default_originate_unset(ospf6, area);
 
+       /* Delete area if no interfaces or configuration. */
+       ospf6_area_no_config_delete(area);
+
        return CMD_SUCCESS;
 }
 

diff --git a/ospf6d/ospf6_area.h b/ospf6d/ospf6_area.h
index d9afd65d81275c14e8eb56daf81781fd9a55c8b9..2ed69cc59785a21b434954ffce86552a57fc079c 100644 (file)
--- a/ospf6d/ospf6_area.h
+++ b/ospf6d/ospf6_area.h
@@ -144,6 +144,7 @@ extern int ospf6_area_cmp(void *va, void *vb);
 extern struct ospf6_area *ospf6_area_create(uint32_t area_id,
                                            struct ospf6 *ospf6, int df);
 extern void ospf6_area_delete(struct ospf6_area *oa);
+extern void ospf6_area_no_config_delete(struct ospf6_area *oa);
 extern struct ospf6_area *ospf6_area_lookup(uint32_t area_id,
                                            struct ospf6 *ospf6);
 extern struct ospf6_area *ospf6_area_lookup_by_area_id(uint32_t area_id);

diff --git a/ospf6d/ospf6_interface.c b/ospf6d/ospf6_interface.c
index 60f92385dd80d14b4fc4d8373ca41909e2cd0d03..1022101eb397778eee94fab508fe4bbd0eb187f3 100644 (file)
--- a/ospf6d/ospf6_interface.c
+++ b/ospf6d/ospf6_interface.c
@@ -1823,6 +1823,9 @@ void ospf6_interface_stop(struct ospf6_interface *oi)
        if (oa->if_list->count == 0) {
                UNSET_FLAG(oa->flag, OSPF6_AREA_ENABLE);
                ospf6_abr_disable_area(oa);
+
+               /* Delete area if no interfaces or configuration. */
+               ospf6_area_no_config_delete(oa);
        }
 }