svn commit: samba r23718 - in branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules: .

[simo]

On Thu, 2007-07-05 at 09:02 +0200, Stefan (metze) Metzmacher wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> abartlet at samba.org schrieb:
> > Author: abartlet
> > Date: 2007-07-05 03:06:59 +0000 (Thu, 05 Jul 2007)
> > New Revision: 23718
> > 
> > WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=23718
> > 
> > Log:
> > Make Samba4 work against the LDAP backend again. 
> > 
> > When we set up the schema, we don't have a partitions container yet.
> > The LDAP error differs from that given by LDB, so I think we still
> > have some conformance work to do.
> 
> yes, the problem is that ldb_tdb returns just 0 search results, but
> success when the basedn isn't present. And it realy should return
> LDB_ERR_NO_SUCH_OBJECT in this case, but we have too many code not
> checking for LDB_ERR_NO_SUCH_OBJECT, so that we would break a lot
> if we would let ldb_tdb return LDB_ERR_NO_SUCH_OBJECT.

I think it is reasonable to change tdb to throw an error if the base
does not exists.
We need to deal with LDB_ERR_NO_SUCH_OBJECT as ldap backends can happily
return that. I will look into this as soon as I can carve some time for
it.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org