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
