On Wed, Apr 16, 2025 at 11:41:31AM +0200, Stefano Brivio wrote:On Wed, 16 Apr 2025 19:07:05 +1000 David Gibson <david(a)gibson.dropbear.id.au> wrote:Sure, done.@@ -239,22 +239,28 @@ static inline void inany_from_af(union inany_addr *aa, /** inany_from_sockaddr - Extract IPv[46] address and port number from sockaddr * @aa: Pointer to store IPv[46] address * @port: Pointer to store port number, host order - * @addr: AF_INET or AF_INET6 socket address + * @addr: Socket addressThis is actually sa_ now but... can we do something for argument names in general, here? What about dst, port, sa, or dst, port, addr?-- David Gibson (he or they) | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you, not the other way | around. http://www.ozlabs.org/~dgibson+ * + * Return: 0 on success, -1 on error (bad address family) */ -static inline void inany_from_sockaddr(union inany_addr *aa, in_port_t *port, - const union sockaddr_inany *sa) +static inline int inany_from_sockaddr(union inany_addr *aa, in_port_t *port, + const void *sa_) { + const union sockaddr_inany *sa = (const union sockaddr_inany *)sa_;