[PATCH] Finish work to enforce smb signing in winbindd

Andrew Bartlett abartlet at samba.org
Thu Oct 16 01:42:50 MDT 2014


On Thu, 2014-10-09 at 07:57 +1300, Andrew Bartlett wrote:
> On Wed, 2014-10-08 at 16:32 +0200, Stefan (metze) Metzmacher wrote:
> > Am 08.10.2014 um 09:27 schrieb Andrew Bartlett:
> > > On Wed, 2014-10-01 at 23:47 +0200, Stefan (metze) Metzmacher wrote:
> > >> Hi Andrew,
> > >>
> > >>> I've tried to look into the failure with your autobuild, but cannot
> > >>> reproduce locally.  But this patch might help at least give us more
> > >>> info.
> > >>
> > >> Yes, 2 private autobuilds also fail.
> > >>
> > >> while a local TDB_NO_FSYNC=1 buildnice make -j test
> > >> TESTS=samba.blackbox.wbinfo FAIL_IMMEDIATELY=1 SOCKET_WRAPPER_KEEP_PCAP=1
> > >> gives unexpected success...
> > > 
> > > One of the challenges is that this test depends on the previous test
> > > calling wbinfo --change-secret.  However, running both doesn't seem to
> > > be enough to trigger it. 
> > > 
> > > Anyway, the attached is enough to fix the branch.
> > > 
> > > I've integrated it with the other patches, and have a new WIP branch
> > > (all the patches) at:
> > > 
> > > https://git.samba.org/abartlet/samba.git/?p=abartlet/samba.git/.git;a=shortlog;h=refs/heads/subdomain-wip4
> > > 
> > > I have the patches for master here:
> > > 
> > > https://git.samba.org/abartlet/samba.git/?p=abartlet/samba.git/.git;a=shortlog;h=refs/heads/subdomain-for-master6
> > > 
> > > The subdomain-wip4 just passed a private autobuild, and the
> > > subdomain-for-master6 branch I'll do a private autobuild on now, but I
> > > expect it to pass as it only strips off unrelated patches. 
> > > 
> > > Please review/push the branch (not the attached fixup demonstration ;-)
> > 
> > I'll look at the branches tomorrow, but the patch seems to introduce
> > a memory leak.
> 
> Thanks!

I've updated the subdomain-for-master6 branch in my repo, and fixed the
memory leak.  I think it is now ready for master.

Please review/push.

Thanks!

-- 
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