vtysh: Show `ip ospf network ...` even if it's not the same as the interface type

ospfv3 shows this unconditionally, and ospfv2 does not show `ip ospf network ...` if the type of the interface matches the specified network.

Fixes: https://github.com/FRRouting/frr/issues/15817
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

diff --git a/ospfd/ospf_interface.c b/ospfd/ospf_interface.c
index b291c248a4b3e8831879dcf13722cafadcf91f31..2ac73828119a4f504f7bcbbdf2bd3c59e64e9be8 100644 (file)
--- a/ospfd/ospf_interface.c
+++ b/ospfd/ospf_interface.c
@@ -1368,7 +1368,8 @@ static int ospf_ifp_create(struct interface *ifp)
            (!OSPF_IF_PARAM_CONFIGURED(IF_DEF_PARAMS(ifp), type) ||
             if_is_loopback(ifp))) {
                SET_IF_PARAM(IF_DEF_PARAMS(ifp), type);
-               IF_DEF_PARAMS(ifp)->type = ospf_default_iftype(ifp);
+               if (!IF_DEF_PARAMS(ifp)->type_cfg)
+                       IF_DEF_PARAMS(ifp)->type = ospf_default_iftype(ifp);
        }
 
        ospf = ifp->vrf->info;

diff --git a/ospfd/ospf_interface.h b/ospfd/ospf_interface.h
index 51fc1bf3c3108c10f4fc426de984c485db4c533f..e5ea08e0e16acf2b34bd4bbf5644eec5d6f688cf 100644 (file)
--- a/ospfd/ospf_interface.h
+++ b/ospfd/ospf_interface.h
@@ -71,6 +71,7 @@ struct ospf_if_params {
        DECLARE_IF_PARAM(struct in_addr, if_area);
        uint32_t if_area_id_fmt;
 
+       bool type_cfg;
        DECLARE_IF_PARAM(uint8_t, type); /* type of interface */
 #define OSPF_IF_ACTIVE                  0
 #define OSPF_IF_PASSIVE                        1

diff --git a/ospfd/ospf_vty.c b/ospfd/ospf_vty.c
index ab9409d1874a990af51fe661528e9da75ae48ebb..e8e9bc2f6c2b759db929ab4ca6ebf8602cf1481d 100644 (file)
--- a/ospfd/ospf_vty.c
+++ b/ospfd/ospf_vty.c
@@ -8457,6 +8457,8 @@ DEFUN(ip_ospf_network, ip_ospf_network_cmd,
            && IF_DEF_PARAMS(ifp)->ptp_dmvpn == old_ptp_dmvpn)
                return CMD_SUCCESS;
 
+       IF_DEF_PARAMS(ifp)->type_cfg = true;
+
        SET_IF_PARAM(IF_DEF_PARAMS(ifp), type);
 
        for (rn = route_top(IF_OIFS(ifp)); rn; rn = route_next(rn)) {
@@ -8506,6 +8508,7 @@ DEFUN (no_ip_ospf_network,
        struct route_node *rn;
 
        IF_DEF_PARAMS(ifp)->type = ospf_default_iftype(ifp);
+       IF_DEF_PARAMS(ifp)->type_cfg = false;
        IF_DEF_PARAMS(ifp)->ptp_dmvpn = 0;
 
        if (IF_DEF_PARAMS(ifp)->type == old_type)
@@ -11854,21 +11857,17 @@ static int config_write_interface_one(struct vty *vty, struct vrf *vrf)
 
                do {
                        /* Interface Network print. */
-                       if (OSPF_IF_PARAM_CONFIGURED(params, type)
-                           && params->type != OSPF_IFTYPE_LOOPBACK) {
-                               if (params->type != ospf_default_iftype(ifp)) {
-                                       vty_out(vty, " ip ospf network %s",
-                                               ospf_int_type_str
-                                                       [params->type]);
-                                       if (params->type
-                                                   == OSPF_IFTYPE_POINTOPOINT
-                                           && params->ptp_dmvpn)
-                                               vty_out(vty, " dmvpn");
-                                       if (params != IF_DEF_PARAMS(ifp) && rn)
-                                               vty_out(vty, " %pI4",
-                                                       &rn->p.u.prefix4);
-                                       vty_out(vty, "\n");
-                               }
+                       if (OSPF_IF_PARAM_CONFIGURED(params, type) &&
+                           params->type != OSPF_IFTYPE_LOOPBACK &&
+                           params->type_cfg) {
+                               vty_out(vty, " ip ospf network %s",
+                                       ospf_int_type_str[params->type]);
+                               if (params->type == OSPF_IFTYPE_POINTOPOINT &&
+                                   params->ptp_dmvpn)
+                                       vty_out(vty, " dmvpn");
+                               if (params != IF_DEF_PARAMS(ifp) && rn)
+                                       vty_out(vty, " %pI4", &rn->p.u.prefix4);
+                               vty_out(vty, "\n");
                        }
 
                        /* OSPF interface authentication print */