dns.idl ==> build breaks on non-ipv6 systems
kai at samba.org
Sun Nov 7 01:58:30 MST 2010
-----BEGIN PGP SIGNED MESSAGE-----
On 2010-11-06 03:05, Michael Adam wrote:
> it seems the introduction of the dns server in s4
> has broken the build on non-ipv6 systems by using
> AD_INET6 unconditionally in the pull/push/print_ipv6
> functions in ndr_basic.c
It's not directly related to the dns server, but I agree I did forget to
put some IPv6 #ifdef around it. In fact, I also got a couple of things
wrong that David pointed out. I'll go and properly fix this.
> Since these functions are in ndr_basic.c this also
> breaks the build of s3 on non-ipv6-systems.
I don't have a system that doesn't have IPv6 support. Is there a box on
the build farm that I can use to check I got the fix right?
> Attached find a patch that moves these 3 functions
> to a new module ndr_ipv6.c that is only used in s4.
I don't know if I like the solution. My plans for the DNS part is to
replace libaddns with an async dns client library that uses dns.idl.
As the IPv6 address is implemented as a basic datatype in PIDL, just the
way IPv4 addresses are, I'd argue the parser code for this into ndr_basic.c
Worldforge developer http://www.worldforge.org/
Wine developer http://wiki.winehq.org/KaiBlin
Samba team member http://www.samba.org/samba/team/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
-----END PGP SIGNATURE-----
More information about the samba-technical