]> git.puffer.fish Git - mirror/frr.git/commitdiff
zebra: Check for NULL in connected_add_ipv6 2069/head
authorDonald Sharp <sharpd@cumulusnetworks.com>
Mon, 16 Apr 2018 15:17:52 +0000 (11:17 -0400)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Mon, 16 Apr 2018 15:17:52 +0000 (11:17 -0400)
When I implemented the same functionality in add_ipv6 that
add_ipv4 has I just assumed that broad would not be NULL with
the ZEBRA_IFA_PEER flag set.

Modify the code to act similiar to the flow of control
in add_ipv4.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
zebra/connected.c

index 35b3b0f4a96df748417d89c8587f6e27680a8847..23f2f666a0593c3b0665714b44098e3d8dc0b857 100644 (file)
@@ -519,12 +519,18 @@ void connected_add_ipv6(struct interface *ifp, int flags, struct in6_addr *addr,
        p->prefixlen = prefixlen;
        ifc->address = (struct prefix *)p;
 
-       if (CHECK_FLAG(ifc->flags, ZEBRA_IFA_PEER)) {
+       if (broad) {
                p = prefix_ipv6_new();
                p->family = AF_INET6;
                IPV6_ADDR_COPY(&p->prefix, broad);
                p->prefixlen = prefixlen;
                ifc->destination = (struct prefix *)p;
+       } else {
+               if (CHECK_FLAG(ifc->flags, ZEBRA_IFA_PEER)) {
+                       zlog_warn("warning: %s called for interface %s with peer flag set, but no peer address supplied",
+                                 __func__, ifp->name);
+                       UNSET_FLAG(ifc->flags, ZEBRA_IFA_PEER);
+               }
        }
 
        /* Label of this address. */