I am not sure if addrstorage was used in this function at one time and was
just never garbage collected. But addrstorage is never declared anywhere
and from the looks of things, it's never read after this field (ss_len)
gets initialized
cheers
diff --git a/src/mhd2/daemon_add_conn.c b/src/mhd2/daemon_add_conn.c
index 437e8de..c26018d 100644
--- a/src/mhd2/daemon_add_conn.c
+++ b/src/mhd2/daemon_add_conn.c
@@ -832,10 +832,6 @@ MHD_daemon_add_connection (struct MHD_Daemon
*MHD_RESTRICT daemon,
"Failed to set noninheritable mode on new client
socket.");
}
-#ifdef HAVE_STRUCT_SOCKADDR_STORAGE_SS_LEN
- addrstorage.ss_len = addrlen; /* Force set the right length */
-#endif /* HAVE_STRUCT_SOCKADDR_STORAGE_SS_LEN */
-
#if defined(MHD_SUPPORT_THREADS)
if (mhd_D_TYPE_HAS_WORKERS (daemon->threading.d_type))
{
