[PATCH 1/2] Fix sin_len/sin6_len handling
Timur I. Bakeyev
timur at freebsd.org
Sun Jul 1 23:24:05 UTC 2018
Hi, James!
You seems did a big job chasing IPv6 address handling. I've recently hit
the same problem, but my fixes were not so extensive...
Couple of questions regarding your patch:
On 1 July 2018 at 21:16, James Clarke via samba-technical <
samba-technical at lists.samba.org> wrote:
> Signed-off-by: James Clarke <jrtc27 at jrtc27.com>
>
> -static bool parse_ipv6(const char *s, const char *ifaces, unsigned port,
> ctdb_sock_addr *saddr)
> +static bool parse_ipv6(const char *s, const char *ifaces, unsigned port,
> struct sockaddr_in6 *sin6)
>
What is the reason and benefit of going from ctdb_sock_addr(which is a
union, that contains struct sockaddr_in6 as well) to struct sockaddr_in6?
> + conf.CHECK_STRUCTURE_MEMBER('struct sockaddr_in6', 'sin6_len',
> + headers='sys/socket.h netinet/in.h',
> + define='HAVE_SOCK_SIN6_LEN')
> +
>
As I'd like this code to work on FreeBSD as well - can you, please, add
'netinet6/in6.h' to the list of the verified headers? It's safe to just add
it to the list - it won't be used if not present in the system.
With best regards,
Timur Bakeyev.
More information about the samba-technical
mailing list