- shutdown(0) for write only sockets

- do not set IPV6_MULTICAST_IF for a read-only socket.
kame · Apr 14, 2000 · 9676ee7 · 9676ee7
1 parent 1ae0c1f
commit 9676ee7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 5 deletions.
diff --git a/kame/kame/dhcp6/dhcp6c.c b/kame/kame/dhcp6/dhcp6c.c
@@ -279,6 +279,11 @@ client6_init()
 		err(1, "setsockopt(outbound, IPV6_MULTICAST_IF)");
 		/*NOTREACHED*/
 	}
+	if (shutdown(outsock, 0)) {
+		/* make the socket write-only */
+		err(1, "shutdown(outbound, 0)");
+		/* NOTREACHED */
+	}
 	freeaddrinfo(res);
 
 #if 0

diff --git a/kame/kame/dhcp6/dhcp6s.c b/kame/kame/dhcp6/dhcp6s.c
@@ -209,11 +209,6 @@ server6_init()
 		err(1, "bind(insock)");
 		/*NOTREACHED*/
 	}
-	if (setsockopt(insock, IPPROTO_IPV6, IPV6_MULTICAST_IF,
-			&ifidx, sizeof(ifidx)) < 0) {
-		err(1, "setsockopt(insock, IPV6_MULTICAST_IF)");
-		/*NOTREACHED*/
-	}
 	freeaddrinfo(res);
 
 	hints.ai_flags = 0;
@@ -261,11 +256,17 @@ server6_init()
 		err(1, "socket(outsock)");
 		/*NOTREACHED*/
 	}
+	/* set outgoing interface of multicast packets for DHCP reconfig */
 	if (setsockopt(outsock, IPPROTO_IPV6, IPV6_MULTICAST_IF,
 			&ifidx, sizeof(ifidx)) < 0) {
 		err(1, "setsockopt(outsock, IPV6_MULTICAST_IF)");
 		/*NOTREACHED*/
 	}
+	if (shutdown(outsock, 0)) {
+		/* make the socket write-only */
+		err(1, "shutdown(outbound, 0)");
+		/* NOTREACHED */
+	}
 	freeaddrinfo(res);
 
 #if 0