if: fix same connected address

Using the same address with a different prefix length is not supported.
If we configure two identical addresses with different
netmasks 192.168.1.1/30 and then 192.168.1.1/29. Zebra sends
'192.168.1.1' with a prefix length of 29. However, the function
'zebra_interface_address_read()' reads '192.168.1.1/30' because the
prefix length is not checked.

Using 'same_prefix()' is more convenient.

Signed-off-by: Loïc Sang <loic.sang@6wind.com>

diff --git a/lib/if.c b/lib/if.c
index a8ceac72432bce03260e903a90a96169d913e115..bf14baaa5e447a5fee3600253227c85f5b6d05fd 100644 (file)
--- a/lib/if.c
+++ b/lib/if.c
@@ -885,21 +885,6 @@ nbr_connected_log(struct nbr_connected *connected, char *str)
        zlog_info("%s", logbuf);
 }
 
-/* If two connected address has same prefix return 1. */
-static int connected_same_prefix(const struct prefix *p1,
-                                const struct prefix *p2)
-{
-       if (p1->family == p2->family) {
-               if (p1->family == AF_INET
-                   && IPV4_ADDR_SAME(&p1->u.prefix4, &p2->u.prefix4))
-                       return 1;
-               if (p1->family == AF_INET6
-                   && IPV6_ADDR_SAME(&p1->u.prefix6, &p2->u.prefix6))
-                       return 1;
-       }
-       return 0;
-}
-
 /* count the number of connected addresses that are in the given family */
 unsigned int connected_count_by_family(struct interface *ifp, int family)
 {
@@ -919,7 +904,7 @@ struct connected *connected_lookup_prefix_exact(struct interface *ifp,
        struct connected *ifc;
 
        frr_each (if_connected, ifp->connected, ifc) {
-               if (connected_same_prefix(ifc->address, p))
+               if (prefix_same(ifc->address, p))
                        return ifc;
        }
        return NULL;
@@ -932,7 +917,7 @@ struct connected *connected_delete_by_prefix(struct interface *ifp,
 
        /* In case of same prefix come, replace it with new one. */
        frr_each_safe (if_connected, ifp->connected, ifc) {
-               if (connected_same_prefix(ifc->address, p)) {
+               if (prefix_same(ifc->address, p)) {
                        if_connected_del(ifp->connected, ifc);
                        return ifc;
                }