summaryrefslogtreecommitdiffstats
path: root/ospfd/ospf_te.c
diff options
context:
space:
mode:
authorRenato Westphal <renato@opensourcerouting.org>2017-10-06 20:25:58 +0200
committerRenato Westphal <renato@opensourcerouting.org>2017-10-10 14:05:46 +0200
commit451fda4f9a2fadc24328e640077780a00ffcdac2 (patch)
treee5cb1b11502a6c525698d188ccc94adfea4515a1 /ospfd/ospf_te.c
parentospf6d: Add missing vrf lookup (diff)
downloadfrr-451fda4f9a2fadc24328e640077780a00ffcdac2.tar.xz
frr-451fda4f9a2fadc24328e640077780a00ffcdac2.zip
*: use the FOR_ALL_INTERFACES abstraction from babeld
This improves code readability and also future-proofs our codebase against new changes in the data structure used to store interfaces. The FOR_ALL_INTERFACES_ADDRESSES macro was also moved to lib/ but for now only babeld is using it. Signed-off-by: Renato Westphal <renato@opensourcerouting.org>
Diffstat (limited to 'ospfd/ospf_te.c')
-rw-r--r--ospfd/ospf_te.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/ospfd/ospf_te.c b/ospfd/ospf_te.c
index 95610e4c7..86f7e7a53 100644
--- a/ospfd/ospf_te.c
+++ b/ospfd/ospf_te.c
@@ -2555,8 +2555,7 @@ DEFUN (show_ip_ospf_mpls_te_link,
if (!ospf->oi_running)
continue;
vrf = vrf_lookup_by_id(ospf->vrf_id);
- RB_FOREACH (ifp, if_name_head,
- &vrf->ifaces_by_name)
+ FOR_ALL_INTERFACES (vrf, ifp)
show_mpls_te_link_sub(vty, ifp);
}
return CMD_SUCCESS;
@@ -2565,7 +2564,7 @@ DEFUN (show_ip_ospf_mpls_te_link,
if (ospf == NULL || !ospf->oi_running)
return CMD_SUCCESS;
vrf = vrf_lookup_by_id(ospf->vrf_id);
- RB_FOREACH (ifp, if_name_head, &vrf->ifaces_by_name)
+ FOR_ALL_INTERFACES (vrf, ifp)
show_mpls_te_link_sub(vty, ifp);
return CMD_SUCCESS;
}
@@ -2575,7 +2574,7 @@ DEFUN (show_ip_ospf_mpls_te_link,
if (!ospf->oi_running)
continue;
vrf = vrf_lookup_by_id(ospf->vrf_id);
- RB_FOREACH (ifp, if_name_head, &vrf->ifaces_by_name)
+ FOR_ALL_INTERFACES (vrf, ifp)
show_mpls_te_link_sub(vty, ifp);
}
}