[PATCH] resolver: do not use stack allocated variables
Matthieu Patou
mat at samba.org
Wed Oct 1 01:36:20 MDT 2014
Volker,
On Tue, Sep 30, 2014 at 08:37:49AM +0200, Volker Lendecke wrote:
> On Mon, Sep 29, 2014 at 05:52:47PM -0700, Matthieu Patou wrote:
> > Hello,
> >
> > Please find the attached patch.
> >
> > Can someone review it ?
>
> It certainly is ok and matches our "normal" calling
> convention. However, in this particular case I see
>
> c->status = nbt_name_dup(state, name, &state->name);
>
> in resolve_name_all_send(). What triggered this patch? Did
> it crash somewhere?
>
> Thanks,
>
> Volker
So the patch dates from last year at IO Lab, we had some strange
behavior with the Windows Samba not able to join 2008R2 domains, Metze
and I found that some DNS records where corrupted and I think that Metze
proposed this patch. So no crash but an incorrect behavior definitely.
Matthieu.
--
Matthieu Patou
Samba Team
http://samba.org
More information about the samba-technical
mailing list