[PATCH] Re: netlogon_creds_cli_validate() in master4-schannel

Andrew Bartlett abartlet at samba.org
Sun Dec 1 13:59:18 MST 2013


On Fri, 2013-11-29 at 18:45 +0100, Stefan (metze) Metzmacher wrote:
> Am 29.11.2013 06:19, schrieb Garming Sam:
> > On 29/11/13 15:19, Andrew Bartlett wrote:
> >> On Fri, 2013-11-29 at 14:50 +1300, Garming Sam wrote:
> >>
> >>>> Segmentation fault (core dumped)
> >>>>
> >>>>
> >>>>
> >>> The seg fault was because it couldn't fetch the secret. This patch
> >>> prints an error in this case and fixes the seg fault by reinitializing
> >>> the pointer.
> >> Thanks Garming!
> >>
> >> Reviewed-by: Andrew Bartlett <abartlet at samba.org>
> >>
> >> Metze: can you review that and add it to you tree?
> >>
> >> Thanks,
> >>
> >> Andrew Bartlett
> >>
> > 
> > Andrew and I just wrote a test to cover the calls that were changed.
> 
> Thanks! I've integrated the patches into my tree, Partly before my changes.

Thanks.  Can you add my review tags to the patches I reviewed last week,
so I don't have to re-tag them?  

I'll also work with Garming to do some manual testing and perhaps add a
few more automated tests. 

Andrew Bartlett

-- 
Andrew Bartlett
http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba






More information about the samba-technical mailing list