acct_ctrl in context_sam_enum_accounts

[Jelmer Vernooij]

On Sun, Sep 29, 2002 at 03:04:25PM +1000, Andrew Bartlett wrote:
> "Stefan (metze) Metzmacher" wrote:
> > 
> > At 18:27 26.09.2002 +0200, Jelmer Vernooij wrote:
> > >Hi all!
> > >
> > >What kind of a value is the acct_ctrl argument of
> > >context_sam_enum_accounts supposed to be?
> > >
> > >It should be possible to list all accounts, that's why the current
> > >implementation isn't very usable...
> > 
> > list all accounts with    acct_ctrl = (ACB_NORMAL | ACB_WSTRUST |
> > ACB_SVRTRUST | ACB_DOMTRUST | ACB_TEMPDUP | ACB_MNS);
> > 
> > I add a patch for this
> 
> I think this is the wrong way to do it.  If the acb_mask is 0, then the
> backend should return all accounts.  Reasoning:  This is how the SAMR
> pipe code behaves, so it makes some sense to do the same.  Otherwise, a
> 'list all accounts' should be just (~0)
> 
> I'll take the rest of this patch however :-)
I already committed this (including the other stuff, so we should probably back that out again)

Jelmer