diff options
| author | Juergen Werner <pogojotz@gmx.net> | 2019-08-13 16:29:09 +0200 |
|---|---|---|
| committer | Juergen Werner <juergen@opensourcerouting.org> | 2019-08-18 18:59:45 +0200 |
| commit | fd267f0808faa24176994ed6d27c095971abdc19 (patch) | |
| tree | 72dba4affd913f8bf21645f4a501530ebfad25f2 /zebra/if_netlink.c | |
| parent | zebra: Do not use connection dest for bcast (diff) | |
| download | frr-fd267f0808faa24176994ed6d27c095971abdc19.tar.xz frr-fd267f0808faa24176994ed6d27c095971abdc19.zip | |
zebra: Correct /32 addr del with broadcast set
Since we are now away from the dual use of the destination field, there
is no need to single out /32 addresses as broadcast. This was bugged
anyway, since the same /32 criteria was used for IPv6 addresses as well,
when `connected_check_ptp` is called in `connected_delete_ipv6`.
Fixes: 3053
Signed-off-by: Juergen Werner <juergen@opensourcerouting.org>
Diffstat (limited to 'zebra/if_netlink.c')
| -rw-r--r-- | zebra/if_netlink.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/zebra/if_netlink.c b/zebra/if_netlink.c index 1f7a1925b..034a23d0a 100644 --- a/zebra/if_netlink.c +++ b/zebra/if_netlink.c @@ -1058,7 +1058,7 @@ int netlink_interface_addr(struct nlmsghdr *h, ns_id_t ns_id, int startup) else connected_delete_ipv4( ifp, flags, (struct in_addr *)addr, - ifa->ifa_prefixlen, (struct in_addr *)broad); + ifa->ifa_prefixlen, NULL); } if (ifa->ifa_family == AF_INET6) { if (ifa->ifa_prefixlen > IPV6_MAX_BITLEN) { @@ -1084,8 +1084,7 @@ int netlink_interface_addr(struct nlmsghdr *h, ns_id_t ns_id, int startup) metric); } else connected_delete_ipv6(ifp, (struct in6_addr *)addr, - (struct in6_addr *)broad, - ifa->ifa_prefixlen); + NULL, ifa->ifa_prefixlen); } return 0; |