More forest trust related patches

Andrew Bartlett abartlet at samba.org
Tue Jun 23 16:57:08 MDT 2015


On Wed, 2015-06-24 at 00:40 +0200, Stefan (metze) Metzmacher wrote:
> Hi Andrew,
> 
> can you have a look at my current master4-forest-ok branch?
> 
> https://git.samba.org/?p=metze/samba/wip.git;a=shortlog;h=refs/heads/master4-forest-ok

I'm in a reviewing mood, so I'll look over that.

> I added more tests and fixed some bugs, which were found.

Great!

> The master4-forest-tmp branch has one more test, which is not completely
> finished.
> (the rpc.lsa test we worked on at SambaXP). I think I just need to fix
> memory leaks
> and remove code that's commented out.
> 
> https://git.samba.org/?p=metze/samba/wip.git;a=shortlog;h=refs/heads/master4-forest-tmp
> 
> So what's left to do before we can push it to master?

I'll work on this now. 

> I can try to add a test regarding the kvno number checking, if it turns out
> to be doable without too much work. But even without such a test I think
> it's pretty clear that the following commit is a required improvement
> compared to the current broken state. See
> https://git.samba.org/?p=metze/samba/wip.git;a=commitdiff;h=bb22983ce51ae41d60ea
> 
> Thanks!
> metze

As you would have seen, it is really, really easy to mess up the KDC
code, but don't worry, I won't hold your patch series ransom to
impossible to write tests :-)

Ideally we would write a unit test framework for the hdb layer like we
did with the bind9 dlz module, as well as the network tests.  (We could
also boil the ocean ;-)

Thanks

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