[PATCH 1/2] Fix sin_len/sin6_len handling
Amitay Isaacs
amitay at gmail.com
Mon Jul 2 00:54:42 UTC 2018
On Mon, Jul 2, 2018 at 9:24 AM, Timur I. Bakeyev via samba-technical
<samba-technical at lists.samba.org> wrote:
> 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.
Thanks for the patches. But we already have these fixes in the pipeline.
Also, tracking the changes to fix the build on freebsd and AIX.
Amitay.
More information about the samba-technical
mailing list