summaryrefslogtreecommitdiff
path: root/source4/lib/ldb/ldb_tdb/ldb_cache.c
diff options
context:
space:
mode:
authorMatthias Dieter Wallnöfer <mwallnoefer@yahoo.de>2008-09-09 17:36:14 +0200
committerJelmer Vernooij <jelmer@samba.org>2008-09-24 19:40:03 +0200
commit9261fa997ccda8f89556b644ddd45cd86919ec2e (patch)
tree7a1d0176a2c339b202d544314575b70d136e6ec8 /source4/lib/ldb/ldb_tdb/ldb_cache.c
parentcf1935817fce0d6c7f4d2797c18f8c116220abd4 (diff)
downloadsamba-9261fa997ccda8f89556b644ddd45cd86919ec2e.tar.gz
samba-9261fa997ccda8f89556b644ddd45cd86919ec2e.tar.bz2
samba-9261fa997ccda8f89556b644ddd45cd86919ec2e.zip
Cosmetic corrections for the LDB backend modules
This commit applies some cosmetic corrections for the LDB backend modules.
Diffstat (limited to 'source4/lib/ldb/ldb_tdb/ldb_cache.c')
-rw-r--r--source4/lib/ldb/ldb_tdb/ldb_cache.c22
1 files changed, 11 insertions, 11 deletions
diff --git a/source4/lib/ldb/ldb_tdb/ldb_cache.c b/source4/lib/ldb/ldb_tdb/ldb_cache.c
index 2576e2c7bd..e502f1e642 100644
--- a/source4/lib/ldb/ldb_tdb/ldb_cache.c
+++ b/source4/lib/ldb/ldb_tdb/ldb_cache.c
@@ -92,11 +92,11 @@ static int ltdb_attributes_flags(struct ldb_message_element *el, unsigned *v)
}
}
if (ltdb_valid_attr_flags[j].name == NULL) {
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
}
*v = value;
- return 0;
+ return LDB_SUCCESS;
}
/*
@@ -118,7 +118,7 @@ static int ltdb_attributes_load(struct ldb_module *module)
goto failed;
}
if (r == LDB_ERR_NO_SUCH_OBJECT) {
- return 0;
+ return LDB_SUCCESS;
}
/* mapping these flags onto ldap 'syntaxes' isn't strictly correct,
but its close enough for now */
@@ -162,9 +162,9 @@ static int ltdb_attributes_load(struct ldb_module *module)
}
}
- return 0;
+ return LDB_SUCCESS;
failed:
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
@@ -258,7 +258,7 @@ int ltdb_cache_load(struct ldb_module *module)
/* a very fast check to avoid extra database reads */
if (ltdb->cache != NULL &&
tdb_get_seqnum(ltdb->tdb) == ltdb->tdb_seqnum) {
- return 0;
+ return LDB_SUCCESS;
}
if (ltdb->cache == NULL) {
@@ -344,7 +344,7 @@ int ltdb_cache_load(struct ldb_module *module)
goto failed;
}
- if (ltdb_attributes_load(module) == -1) {
+ if (ltdb_attributes_load(module) != LDB_SUCCESS) {
goto failed;
}
@@ -353,14 +353,14 @@ done:
talloc_free(baseinfo);
talloc_free(baseinfo_dn);
talloc_free(indexlist_dn);
- return 0;
+ return LDB_SUCCESS;
failed:
talloc_free(options);
talloc_free(baseinfo);
talloc_free(baseinfo_dn);
talloc_free(indexlist_dn);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
@@ -449,10 +449,10 @@ int ltdb_check_at_attributes_values(const struct ldb_val *value)
for (i = 0; ltdb_valid_attr_flags[i].name != NULL; i++) {
if ((strcmp(ltdb_valid_attr_flags[i].name, (char *)value->data) == 0)) {
- return 0;
+ return LDB_SUCCESS;
}
}
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}