Mercurial > hg > dhcpcd

changeset 1806:725417fae1e5 draft

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Avoid sillyness if not a valid IPv6 address
author Roy Marples <roy@marples.name>
date Sat, 24 Nov 2012 20:36:20 +0000
parents b3ac4d025b78
children d2a966e2a741
files ipv6rs.c
diffstat 1 files changed, 22 insertions(+), 10 deletions(-) [+]
line wrap: on
line diff
--- a/ipv6rs.c	Sat Nov 24 20:17:33 2012 +0000
+++ b/ipv6rs.c	Sat Nov 24 20:36:20 2012 +0000
@@ -378,7 +378,7 @@
 void
 ipv6rs_handledata(_unused void *arg)
 {
-	ssize_t len, l, n, olen;
+	ssize_t len, l, m, n, olen;
 	struct cmsghdr *cm;
 	int hoplimit;
 	struct in6_pktinfo pkt;
@@ -632,22 +632,34 @@
 			for (n = ndo->nd_opt_len - 1; n > 1; n -= 2,
 			    op += sizeof(addr.s6_addr))
 			{
-				l += ipv6_printaddr(NULL, 0, op, ifp->name) + 1;
+				m = ipv6_printaddr(NULL, 0, op, ifp->name);
+				if (m != -1)
+					l += m + 1;
 			}
 			op = (uint8_t *)ndo;
 			op += offsetof(struct nd_opt_rdnss,
 			    nd_opt_rdnss_lifetime);
 			op += sizeof(rdnss->nd_opt_rdnss_lifetime);
 			tmp = opt = malloc(l);
-			for (n = ndo->nd_opt_len - 1; n > 1; n -= 2,
-			    op += sizeof(addr.s6_addr))
-			{
-				tmp += ipv6_printaddr(tmp, l, op, ifp->name);
-				*tmp++ = ' ';
-				if (lifetime > 0)
-					has_dns = 1;
+			if (opt) {
+				for (n = ndo->nd_opt_len - 1; n > 1; n -= 2,
+				    op += sizeof(addr.s6_addr))
+				{
+					m = ipv6_printaddr(tmp, l, op,
+					    ifp->name);
+					if (m != -1) {
+						l -= (m + 1);
+						tmp += m;
+						*tmp++ = ' ';
+						if (lifetime > 0)
+							has_dns = 1;
+					}
+				}
+				if (tmp != opt)
+					(*--tmp) = '\0';
+				else
+					*opt = '\0';
 			}
-			(*--tmp) = '\0';
 			break;
 			
 		case ND_OPT_DNSSL: