From d60977cc7f89f89f34187f310c91d1ab7db6ccf2 Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Thu, 11 Sep 2008 20:51:26 +1000 Subject: [PATCH] Remove the complexity of transactions from the attributes-setting code. I think it is just too complex and error prone to init and cancel transactions during the module init code. Instead, this isn't prone to races as it will always achieve a steady state (eventually), and most cases will never do the write. Andrew Bartlett --- source/dsdb/schema/schema_set.c | 17 +---------------- 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/source/dsdb/schema/schema_set.c b/source/dsdb/schema/schema_set.c index 0ca26c0fc71..3f5ddd8446e 100644 --- a/source/dsdb/schema/schema_set.c +++ b/source/dsdb/schema/schema_set.c @@ -115,24 +115,17 @@ static int dsdb_schema_set_attributes(struct ldb_context *ldb, struct dsdb_schem } - ret = ldb_transaction_start(ldb); - if (ret != LDB_SUCCESS) { - return ret; - } - /* Try to avoid churning the attributes too much - we only want to do this if they have changed */ ret = ldb_search_exp_fmt(ldb, mem_ctx, &res, msg->dn, LDB_SCOPE_BASE, NULL, "dn=%s", ldb_dn_get_linearized(msg->dn)); if (ret == LDB_ERR_NO_SUCH_OBJECT) { ret = ldb_add(ldb, msg); } else if (ret != LDB_SUCCESS) { talloc_free(mem_ctx); - ldb_transaction_cancel(ldb); return ret; } else { if (res->count != 1) { talloc_free(mem_ctx); - ldb_transaction_cancel(ldb); return LDB_ERR_NO_SUCH_OBJECT; } @@ -149,10 +142,8 @@ static int dsdb_schema_set_attributes(struct ldb_context *ldb, struct dsdb_schem if (ret == LDB_ERR_INSUFFICIENT_ACCESS_RIGHTS) { /* We might be on a read-only DB */ talloc_free(mem_ctx); - ret = ldb_transaction_cancel(ldb); return ret; } else if (ret != LDB_SUCCESS) { - ldb_transaction_cancel(ldb); return ret; } @@ -163,12 +154,10 @@ static int dsdb_schema_set_attributes(struct ldb_context *ldb, struct dsdb_schem ret = ldb_add(ldb, msg_idx); } else if (ret != LDB_SUCCESS) { talloc_free(mem_ctx); - ldb_transaction_cancel(ldb); return ret; } else { if (res_idx->count != 1) { talloc_free(mem_ctx); - ldb_transaction_cancel(ldb); return LDB_ERR_NO_SUCH_OBJECT; } @@ -183,11 +172,7 @@ static int dsdb_schema_set_attributes(struct ldb_context *ldb, struct dsdb_schem if (ret == LDB_ERR_INSUFFICIENT_ACCESS_RIGHTS) { /* We might be on a read-only DB */ talloc_free(mem_ctx); - return ldb_transaction_cancel(ldb); - } else if (ret == LDB_SUCCESS) { - ret = ldb_transaction_commit(ldb); - } else { - ldb_transaction_cancel(ldb); + ret = LDB_SUCCESS; } talloc_free(mem_ctx); return ret; -- 2.34.1