r15978: - pass the error code back to the caller...

- we were giving OPERATIONS_ERROR in all cases:-(
- we now pass ALREADY_EXIST fine to the caller,
  and the code in libnet_site.c is happy again.
- this bug wasn't noticed for a long time because
  the ldb_ildap code always passed SUCCESS to it's caller

metze
(This used to be commit 7b9d7119205c442f338deab07122ce1548bad9b6)

diff --git a/source4/lib/ldb/ldb_tdb/ldb_tdb.c b/source4/lib/ldb/ldb_tdb/ldb_tdb.c
index 629a18b9c67b7a4806fa1d4ee2313786ea0690f5..29c2703644a727042523bd317ef33a56c9555bc0 100644 (file)
--- a/source4/lib/ldb/ldb_tdb/ldb_tdb.c
+++ b/source4/lib/ldb/ldb_tdb/ldb_tdb.c
@@ -262,7 +262,7 @@ static int ltdb_add_internal(struct ldb_module *module, const struct ldb_message
        
        ret = ltdb_check_special_dn(module, msg);
        if (ret != LDB_SUCCESS) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return ret;
        }
        
        if (ltdb_cache_load(module) != 0) {
@@ -271,7 +271,7 @@ static int ltdb_add_internal(struct ldb_module *module, const struct ldb_message
 
        ret = ltdb_store(module, msg, TDB_INSERT);
        if (ret != LDB_SUCCESS) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return ret;
        }
 
        ret = ltdb_modified(module, msg->dn);