X-Git-Url: https://git.decadent.org.uk/gitweb/?a=blobdiff_plain;f=src%2Fra.c;h=b249288279955446ecf59e061a4015b0314b3302;hb=dcbce9858dabb558980c537ff2332ff8f25e740e;hp=2149a66b7ce929b81fc7bf44e4ab373d9ccead7d;hpb=29937f316f27e9ead2f8a0d9cb419aa810fa9d02;p=odhcp6c.git

diff --git a/src/ra.c b/src/ra.c
index 2149a66..b249288 100644
--- a/src/ra.c
+++ b/src/ra.c
@@ -125,7 +125,7 @@ static bool ra_deduplicate(const struct in6_addr *any, uint8_t length)
 {
 	struct odhcp6c_entry entry = {IN6ADDR_ANY_INIT, length, 0, *any, 0, 0};
 	struct odhcp6c_entry *x = odhcp6c_find_entry(STATE_RA_PREFIX, &entry);
-	if (x) {
+	if (x && IN6_ARE_ADDR_EQUAL(&x->target, any)) {
 		odhcp6c_random(&x->target.s6_addr32[2], 2 * sizeof(uint32_t));
 	} else if (odhcp6c_find_entry(STATE_IA_NA, &entry)) {
 		dhcpv6_request(DHCPV6_MSG_DECLINE);
@@ -149,15 +149,15 @@ bool ra_rtnl_process(void)
 					nh = NLMSG_NEXT(nh, len)) {
 			struct ifaddrmsg *ifa = NLMSG_DATA(nh);
 			struct in6_addr *addr = NULL;
-			if (nh->nlmsg_type != RTM_NEWADDR || NLMSG_PAYLOAD(nh, 0) < sizeof(*ifa) ||
-					!(ifa->ifa_flags & IFA_F_DADFAILED) ||
-					ifa->ifa_index != if_index)
+			if (NLMSG_PAYLOAD(nh, 0) < sizeof(*ifa) || ifa->ifa_index != if_index ||
+					(nh->nlmsg_type == RTM_NEWADDR && !(ifa->ifa_flags & IFA_F_DADFAILED)) ||
+					(nh->nlmsg_type == RTM_DELADDR && !(ifa->ifa_flags & IFA_F_TENTATIVE)))
 				continue;
 
 			ssize_t alen = NLMSG_PAYLOAD(nh, sizeof(*ifa));
 			for (struct rtattr *rta = (struct rtattr*)&ifa[1]; RTA_OK(rta, alen);
 					rta = RTA_NEXT(rta, alen))
-				if (rta->rta_type == IFA_LOCAL && RTA_PAYLOAD(rta) >= sizeof(*addr))
+				if (rta->rta_type == IFA_ADDRESS && RTA_PAYLOAD(rta) >= sizeof(*addr))
 					addr = RTA_DATA(rta);
 
 			if (addr)
@@ -275,7 +275,7 @@ bool ra_process(void)
 
 		size_t ra_dns_len;
 		struct odhcp6c_entry *entry = odhcp6c_get_state(STATE_RA_DNS, &ra_dns_len);
-		for (size_t i = 0; i < len / sizeof(*entry); ++i)
+		for (size_t i = 0; i < ra_dns_len / sizeof(*entry); ++i)
 			if (IN6_ARE_ADDR_EQUAL(&entry[i].router, &from.sin6_addr) &&
 					entry[i].valid > router_valid)
 				entry[i].valid = router_valid;