zebra: improve identification of loopback interfaces in the router-id code

The if_is_loopback() function is the right abstraction for identifying
loopback interfaces. There should be no reason for not using it in the
router-id code.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>
This commit is contained in:
Renato Westphal 2018-11-17 21:33:37 -02:00
parent fef6daf4da
commit 106935f671

View File

@ -132,8 +132,7 @@ void router_id_add_address(struct connected *ifc)
router_id_get(&before, zvrf_id(zvrf));
if (!strncmp(ifc->ifp->name, "lo", 2)
|| !strncmp(ifc->ifp->name, "dummy", 5))
if (if_is_loopback(ifc->ifp))
l = zvrf->rid_lo_sorted_list;
else
l = zvrf->rid_all_sorted_list;
@ -165,8 +164,7 @@ void router_id_del_address(struct connected *ifc)
router_id_get(&before, zvrf_id(zvrf));
if (!strncmp(ifc->ifp->name, "lo", 2)
|| !strncmp(ifc->ifp->name, "dummy", 5))
if (if_is_loopback(ifc->ifp))
l = zvrf->rid_lo_sorted_list;
else
l = zvrf->rid_all_sorted_list;