This commit applies some cosmetic corrections for the LDB backend modules.
module = talloc(ldb, struct ldb_module);
if (!module) {
ldb_oom(ldb);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
talloc_set_name_const(module, "ldb_ildap backend");
module->ldb = ldb;
}
*_module = module;
- return 0;
+ return LDB_SUCCESS;
failed:
talloc_free(module);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
_PUBLIC_ const struct ldb_backend_ops ldb_ldap_backend_ops = {
count = ldap_count_values_len(bval);
if (count <= 0) {
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
el = talloc_realloc(msg, msg->elements, struct ldb_message_element,
msg->num_elements + 1);
if (!el) {
errno = ENOMEM;
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
msg->elements = el;
el->name = talloc_strdup(msg->elements, attr);
if (!el->name) {
errno = ENOMEM;
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
el->flags = 0;
el->values = talloc_array(msg->elements, struct ldb_val, count);
if (!el->values) {
errno = ENOMEM;
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
for (i=0;i<count;i++) {
el->values[i].data = talloc_size(el->values, bval[i]->bv_len+1);
if (!el->values[i].data) {
errno = ENOMEM;
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
memcpy(el->values[i].data, bval[i]->bv_val, bval[i]->bv_len);
el->values[i].data[bval[i]->bv_len] = 0;
msg->num_elements++;
- return 0;
+ return LDB_SUCCESS;
}
/*
if (module == NULL) {
ldb_oom(ldb);
talloc_free(lldb);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
talloc_set_name_const(module, "ldb_ldap backend");
module->ldb = ldb;
}
*_module = module;
- return 0;
+ return LDB_SUCCESS;
failed:
talloc_free(module);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
const struct ldb_backend_ops ldb_ldap_backend_ops = {
printf("lsqlite3_safe_rollback: Error: %s\n", errmsg);
free(errmsg);
}
- return -1;
+ return SQLITE_ERROR;
}
- return 0;
+ return SQLITE_OK;
}
/* return an eid as result */
printf("lsqlite3_start_trans: error: %s\n", errmsg);
free(errmsg);
}
- return -1;
+ return SQLITE_ERROR;
}
};
lsqlite3->trans_count++;
- return 0;
+ return SQLITE_OK;
}
static int lsql_end_trans(struct ldb_module *module)
if (lsqlite3->trans_count > 0) {
lsqlite3->trans_count--;
- } else return -1;
+ } else return SQLITE_ERROR;
if (lsqlite3->trans_count == 0) {
ret = sqlite3_exec(lsqlite3->sqlite, "COMMIT;", NULL, NULL, &errmsg);
printf("lsqlite3_end_trans: error: %s\n", errmsg);
free(errmsg);
}
- return -1;
+ return SQLITE_ERROR;
}
}
- return 0;
+ return SQLITE_OK;
}
static int lsql_del_trans(struct ldb_module *module)
if (lsqlite3->trans_count > 0) {
lsqlite3->trans_count--;
- } else return -1;
+ } else return SQLITE_ERROR;
if (lsqlite3->trans_count == 0) {
return lsqlite3_safe_rollback(lsqlite3->sqlite);
}
- return -1;
+ return SQLITE_ERROR;
}
static int destructor(struct lsqlite3_private *lsqlite3)
if (lsqlite3->sqlite) {
sqlite3_close(lsqlite3->sqlite);
}
- return 0;
+ return SQLITE_OK;
}
static int lsql_request(struct ldb_module *module, struct ldb_request *req)
/* create a local ctx */
local_ctx = talloc_named(lsqlite3, 0, "lsqlite3_rename local context");
if (local_ctx == NULL) {
- return -1;
+ return SQLITE_ERROR;
}
schema = lsqlite3_tprintf(local_ctx,
failed:
if (rollback) lsqlite3_safe_rollback(lsqlite3->sqlite);
sqlite3_close(lsqlite3->sqlite);
- return -1;
+ return SQLITE_ERROR;
}
/*
}
}
- return 0;
+ return LDB_SUCCESS;
failed:
if (lsqlite3->sqlite != NULL) {
(void) sqlite3_close(lsqlite3->sqlite);
}
talloc_free(lsqlite3);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
const struct ldb_backend_ops ldb_sqlite3_backend_ops = {
}
}
if (ltdb_valid_attr_flags[j].name == NULL) {
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
}
*v = value;
- return 0;
+ return LDB_SUCCESS;
}
/*
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 */
}
}
- return 0;
+ return LDB_SUCCESS;
failed:
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
/* 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) {
goto failed;
}
- if (ltdb_attributes_load(module) == -1) {
+ if (ltdb_attributes_load(module) != LDB_SUCCESS) {
goto failed;
}
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;
}
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;
}
dn = ldb_dn_get_linearized(message->dn);
if (dn == NULL) {
errno = ENOMEM;
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
/* work out how big it needs to be */
data->dptr = talloc_array(ldb, uint8_t, size);
if (!data->dptr) {
errno = ENOMEM;
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
data->dsize = size;
}
}
- return 0;
+ return LDB_SUCCESS;
}
/*
if (message->num_elements == 0) {
message->elements = NULL;
- return 0;
+ return LDB_SUCCESS;
}
if (message->num_elements > remaining / 6) {
"Error: %d bytes unread in ltdb_unpack_data\n", remaining);
}
- return 0;
+ return LDB_SUCCESS;
failed:
talloc_free(message->elements);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
ret = ltdb_unpack_data(module, &tdb_data, msg);
free(tdb_data.dptr);
- if (ret == -1) {
+ if (ret != LDB_SUCCESS) {
return LDB_ERR_OPERATIONS_ERROR;
}
if (ltdb->in_transaction == 0) {
return tdb_lockall_read(ltdb->tdb);
}
- return 0;
+ return LDB_SUCCESS;
}
/*
if (ltdb->in_transaction == 0) {
return tdb_unlockall_read(ltdb->tdb);
}
- return 0;
+ return LDB_SUCCESS;
}
/*
/* pull the attributes that the user wants */
msg2 = ltdb_pull_attrs(module, mem_ctx, msg, attrs);
if (!msg2) {
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
/* add to the results list */
res2 = talloc_realloc(mem_ctx, *res, struct ldb_message *, (*count)+2);
if (!res2) {
talloc_free(msg2);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
(*res) = res2;
(*res)[(*count)+1] = NULL;
(*count)++;
- return 0;
+ return LDB_SUCCESS;
}
if (ldb_attr_cmp(attrs[i], "distinguishedName") == 0) {
if (msg_add_distinguished_name(msg) != 0) {
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
}
}
if (keep_all) {
if (msg_add_distinguished_name(msg) != 0) {
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
- return 0;
+ return LDB_SUCCESS;
}
for (i = 0; i < msg->num_elements; i++) {
}
}
- return 0;
+ return LDB_SUCCESS;
}
/*
if (key.dsize < 4 ||
strncmp((char *)key.dptr, "DN=", 3) != 0) {
- return 0;
+ return LDB_SUCCESS;
}
ares = talloc_zero(ac, struct ldb_reply);
if (!ares) {
handle->status = LDB_ERR_OPERATIONS_ERROR;
handle->state = LDB_ASYNC_DONE;
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
ares->message = ldb_msg_new(ares);
handle->status = LDB_ERR_OPERATIONS_ERROR;
handle->state = LDB_ASYNC_DONE;
talloc_free(ares);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
/* unpack the record */
ret = ltdb_unpack_data(ac->module, &data, ares->message);
- if (ret == -1) {
+ if (ret) {
talloc_free(ares);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
if (!ares->message->dn) {
handle->status = LDB_ERR_OPERATIONS_ERROR;
handle->state = LDB_ASYNC_DONE;
talloc_free(ares);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
}
if (!ldb_match_msg(ac->module->ldb, ares->message, ac->tree,
ac->base, ac->scope)) {
talloc_free(ares);
- return 0;
+ return LDB_SUCCESS;
}
/* filter the attributes that the user wants */
ret = ltdb_filter_attrs(ares->message, ac->attrs);
- if (ret == -1) {
+ if (ret != LDB_SUCCESS) {
handle->status = LDB_ERR_OPERATIONS_ERROR;
handle->state = LDB_ASYNC_DONE;
talloc_free(ares);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
ares->type = LDB_REPLY_ENTRY;
if (handle->status != LDB_SUCCESS) {
/* don't try to free ares here, the callback is in charge of that */
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
- return 0;
+ return LDB_SUCCESS;
}
if (! ldb_dn_is_special(msg->dn) ||
! ldb_dn_check_special(msg->dn, LTDB_ATTRIBUTES)) {
- return 0;
+ return LDB_SUCCESS;
}
/* we have @ATTRIBUTES, let's check attributes are fine */
}
}
- return 0;
+ return LDB_SUCCESS;
}
}
ret = ltdb_pack_data(module, msg, &tdb_data);
- if (ret == -1) {
+ if (ret != LDB_SUCCESS) {
talloc_free(tdb_key.dptr);
return LDB_ERR_OTHER;
}
}
ret = ltdb_unpack_data(module, &tdb_data, msg2);
- if (ret == -1) {
+ if (ret != LDB_SUCCESS) {
ret = LDB_ERR_OTHER;
goto failed;
}
if (strncmp(url, "tdb://", 6) != 0) {
ldb_debug(ldb, LDB_DEBUG_ERROR,
"Invalid tdb URL '%s'", url);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
path = url+6;
} else {
ltdb = talloc_zero(ldb, struct ltdb_private);
if (!ltdb) {
ldb_oom(ldb);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
/* note that we use quite a large default hash size */
ldb_debug(ldb, LDB_DEBUG_ERROR,
"Unable to open tdb '%s'\n", path);
talloc_free(ltdb);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
ltdb->sequence_number = 0;
if (!module) {
ldb_oom(ldb);
talloc_free(ltdb);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
talloc_set_name_const(*module, "ldb_tdb backend");
(*module)->ldb = ldb;
if (ltdb_cache_load(*module) != 0) {
talloc_free(*module);
talloc_free(ltdb);
- return -1;
+ return LDB_ERR_OPERATIONS_ERROR;
}
- return 0;
+ return LDB_SUCCESS;
}
const struct ldb_backend_ops ldb_tdb_backend_ops = {
if (w == tdb_list) {
tdb_list = w->next;
}
- return 0;
+ return LDB_SUCCESS;
}
static void ltdb_log_fn(struct tdb_context *tdb, enum tdb_debug_level level, const char *fmt, ...) PRINTF_ATTRIBUTE(3, 4);