resolv_wrapper and __res functions (was: Re: [PATCH 0/5] dns: Add authority record to replies)
asn at samba.org
Mon Aug 10 08:31:47 UTC 2015
On Monday 10 August 2015 14:22:11 Andrew Bartlett wrote:
> On Wed, 2015-08-05 at 08:48 +0200, Kai Blin wrote:
> > On 30/07/15 13:24, Kai Blin wrote:
> > Ok, after some more digging..
> > > But also it looks like nsupdate keeps sending SOA requests and not
> > > liking our reply. I'll investigate a bit more to find out what the
> > > heck
> > > it's expecting.
> > Something between `make testenv` and calling nsupdate with
> > SOCKETWRAPPER
> > seems to be misplacing the UDP packets from the server. Both
> > `nsupdate`
> > as well as `nsupdate -g` work just fine against an actual Samba AD DC
> > provisioned outside our test environment. Ok, `nsupdate -g` complains
> > about the TSIG coming from the server, but that's an unrelated bug.
> > The PCAP file captured by socketwrapper in `make testenv` also looks
> > identical to what happens on the wire for a real system, but
> > `nsupdate`
> > still keeps resending the SOA request and not seeing the reply until
> > it
> > gives up claiming the name server could not be reached.
> > So, seeing how that is an unrelated issue, can these patches go in
> > now?
That nsupdate doesn't work is a bug in socket_wrapper and a bug in the bind
low level functions. I've fixed the socket_wrapper bug here:
A test is missing to push it upstream. I'm working on that.
I've reported the bind bug here:
> I've reproduced the same thing. I needed this patch (or at least the
> __res_nquery part) to get that far however, as otherwise resolv_wrapper
> was exiting the samba_dnsupdate binary with 255, unable to find
> res_nquery when that has been defined aside to __res_nquery.
Andrew, could you please provide a patch the resolv_wrapper repository:
Andreas Schneider GPG-ID: CC014E3D
Samba Team asn at samba.org
More information about the samba-technical