[Fwd: Re: Samba 4 libnet_join and RPC-JOIN torture test]

Andrew Bartlett abartlet at samba.org
Fri Sep 23 13:24:58 GMT 2005


On Fri, 2005-09-23 at 11:36 +0200, Stefan Metzmacher wrote:
> Am Montag, 19. September 2005 16:19 schrieb Brad Henry:
> > Hi metze, have you had a chance to look over this patch? Besides the
> > error handling, is there anything you think I need to do to make this
> > acceptable for Samba 4?
> Hi Brad, Hi Andrew,
> 
> that patch was quite good, but had a few little cut'n'paste bugs...
> 
> I have changed this:
> - a few typos
> - using talloc_steal(mem_ctx, ...) instead of talloc_steal(r, ...) because we
>    can't assume that mem_ctx and r are the same!

This was actually deliberate, it was intentional (for better coding
style) that the caller would need to make 'r' talloc()ed.  

> - make libnet_JoinADSDomain and libnet_JoinSite static
> - remove schema_dn_str and ntds_dn_str from JoinSite() becuase they're not
>   used yet.
> - remove white spaces and use tabs in the torture/ code
> 
> if you're both happy with this patch:
> Andrew: please to some RPC-JOIN tests and net join runtime tests, and commits 
> this patch

Yep, I'll test and get this in.

Andrew Bartlett

-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Samba Developer, SuSE Labs, Novell Inc.        http://suse.de
Authentication Developer, Samba Team           http://samba.org
Student Network Administrator, Hawker College  http://hawkerc.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.samba.org/archive/samba-technical/attachments/20050923/e2805e39/attachment.bin


More information about the samba-technical mailing list