commit 14aa6a729393403e45e3c78a2224d1c323fe0c06
from: Stefan Sperling <stsp@stsp.name>
date: Sat Aug 06 07:41:29 2022 UTC

remove the last remaining use of sin_len from gotwebd

I don't think this assignment to sin_len is needed at all, because
sin_len is already being set via got_sockaddr_inet[6]_init() during
the configuration stage.

Tested successfully on both fcgi socket types (unix / tcp).

commit - 6a5ff2d4f6001aa22007d025482d4cd6109a6b0d
commit + 14aa6a729393403e45e3c78a2224d1c323fe0c06
blob - b8e00918c13b3d92a2fe2fb2769ca40d82a8bec6
blob + 0b020ab754c33ba525068d380e871877d551f8e5
--- gotwebd/sockets.c
+++ gotwebd/sockets.c
@@ -75,7 +75,6 @@ void	 sockets_rlimit(int);
 int	 sockets_dispatch_gotwebd(int, struct privsep_proc *, struct imsg *);
 int	 sockets_unix_socket_listen(struct privsep *, struct socket *);
 int	 sockets_create_socket(struct addresslist *, in_port_t);
-int	 sockets_socket_af(struct sockaddr_storage *, in_port_t);
 int	 sockets_accept_reserve(int, struct sockaddr *, socklen_t *, int,
 	    volatile int *);
 
@@ -290,27 +289,6 @@ done:
 	return (sock);
 }
 
-int
-sockets_socket_af(struct sockaddr_storage *ss, in_port_t port)
-{
-	switch (ss->ss_family) {
-	case AF_INET:
-		((struct sockaddr_in *)ss)->sin_port = port;
-		((struct sockaddr_in *)ss)->sin_len =
-		    sizeof(struct sockaddr_in);
-		break;
-	case AF_INET6:
-		((struct sockaddr_in6 *)ss)->sin6_port = port;
-		((struct sockaddr_in6 *)ss)->sin6_len =
-		    sizeof(struct sockaddr_in6);
-		break;
-	default:
-		return -1;
-	}
-
-	return 0;
-}
-
 void
 sockets_launch(void)
 {
@@ -558,8 +536,15 @@ sockets_create_socket(struct addresslist *al, in_port_
 	hints.ai_flags |= AI_PASSIVE;
 
 	TAILQ_FOREACH(a, al, entry) {
-		if (sockets_socket_af(&a->ss, port) == -1) {
-			log_warnx("%s: sockets_socket_af", __func__);
+		switch (a->ss.ss_family) {
+		case AF_INET:
+			((struct sockaddr_in *)(&a->ss))->sin_port = port;
+			break;
+		case AF_INET6:
+			((struct sockaddr_in6 *)(&a->ss))->sin6_port = port;
+			break;
+		default:
+			log_warnx("%s: unknown address family", __func__);
 			goto fail;
 		}