r26135: Remove samdb_add(), samdb_delete() and samdb_modify(), which were just

wrappers to ldb_add() etc.  samdb_replace() remains, as it sets flags on
all entries as 'replace'.

Andrew Bartlett
(This used to be commit 09c0faa5b7e1a560bf13b99a2584012a47377bb6)

3 files changed, 13 insertions, 12 deletions

diff --git a/source4/rpc_server/drsuapi/dcesrv_drsuapi.c b/source4/rpc_server/drsuapi/dcesrv_drsuapi.c
index 21c4d36e9b..a0663a588d 100644
--- a/source4/rpc_server/drsuapi/dcesrv_drsuapi.c
+++ b/source4/rpc_server/drsuapi/dcesrv_drsuapi.c
@@ -431,7 +431,7 @@ static WERROR dcesrv_drsuapi_DsWriteAccountSpn(struct dcesrv_call_state *dce_cal
    
 			/* Apply to database */
 
-			ret = samdb_modify(b_state->sam_ctx, mem_ctx, msg);
+			ret = ldb_modify(b_state->sam_ctx, msg);
 			if (ret != 0) {
 				DEBUG(0,("Failed to modify SPNs on %s: %s\n",
 					 ldb_dn_get_linearized(msg->dn), 
diff --git a/source4/rpc_server/lsa/dcesrv_lsa.c b/source4/rpc_server/lsa/dcesrv_lsa.c
index 0b426145c6..93c1ee9e4d 100644
--- a/source4/rpc_server/lsa/dcesrv_lsa.c
+++ b/source4/rpc_server/lsa/dcesrv_lsa.c
@@ -100,7 +100,8 @@ static NTSTATUS dcesrv_lsa_Delete(struct dcesrv_call_state *dce_call, TALLOC_CTX
 	DCESRV_PULL_HANDLE(h, r->in.handle, DCESRV_HANDLE_ANY);
 	if (h->wire_handle.handle_type == LSA_HANDLE_SECRET) {
 		struct lsa_secret_state *secret_state = h->data;
-		ret = samdb_delete(secret_state->sam_ldb, mem_ctx, secret_state->secret_dn);
+		ret = ldb_delete(secret_state->sam_ldb, 
+				 secret_state->secret_dn);
 		talloc_free(h);
 		if (ret != 0) {
 			return NT_STATUS_INVALID_HANDLE;
@@ -109,8 +110,8 @@ static NTSTATUS dcesrv_lsa_Delete(struct dcesrv_call_state *dce_call, TALLOC_CTX
 		return NT_STATUS_OK;
 	} else if (h->wire_handle.handle_type == LSA_HANDLE_TRUSTED_DOMAIN) {
 		struct lsa_trusted_domain_state *trusted_domain_state = h->data;
-		ret = samdb_delete(trusted_domain_state->policy->sam_ldb, mem_ctx, 
-				   trusted_domain_state->trusted_domain_dn);
+		ret = ldb_delete(trusted_domain_state->policy->sam_ldb, 
+				 trusted_domain_state->trusted_domain_dn);
 		talloc_free(h);
 		if (ret != 0) {
 			return NT_STATUS_INVALID_HANDLE;
@@ -1478,7 +1479,7 @@ static NTSTATUS dcesrv_lsa_AddRemoveAccountRights(struct dcesrv_call_state *dce_
 		return NT_STATUS_OK;
 	}
 
-	ret = samdb_modify(state->sam_ldb, mem_ctx, msg);
+	ret = ldb_modify(state->sam_ldb, msg);
 	if (ret != 0) {
 		if (ldb_flag == LDB_FLAG_MOD_DELETE && ret == LDB_ERR_NO_SUCH_ATTRIBUTE) {
 			return NT_STATUS_OBJECT_NAME_NOT_FOUND;
@@ -1747,7 +1748,7 @@ static NTSTATUS dcesrv_lsa_CreateSecret(struct dcesrv_call_state *dce_call, TALL
 	secret_state->secret_dn = talloc_reference(secret_state, msg->dn);
 
 	/* create the secret */
-	ret = samdb_add(secret_state->sam_ldb, mem_ctx, msg);
+	ret = ldb_add(secret_state->sam_ldb, msg);
 	if (ret != 0) {
 		DEBUG(0,("Failed to create secret record %s: %s\n",
 			 ldb_dn_get_linearized(msg->dn), 
diff --git a/source4/rpc_server/samr/dcesrv_samr.c b/source4/rpc_server/samr/dcesrv_samr.c
index e0d965da9e..9d208f8444 100644
--- a/source4/rpc_server/samr/dcesrv_samr.c
+++ b/source4/rpc_server/samr/dcesrv_samr.c
@@ -1021,7 +1021,7 @@ static NTSTATUS dcesrv_samr_CreateDomainGroup(struct dcesrv_call_state *dce_call
 	samdb_msg_add_string(d_state->sam_ctx, mem_ctx, msg, "objectClass", "group");
 			     
 	/* create the group */
-	ret = samdb_add(d_state->sam_ctx, mem_ctx, msg);
+	ret = ldb_add(d_state->sam_ctx, msg);
 	switch (ret) {
 	case  LDB_SUCCESS:
 		break;
@@ -1303,7 +1303,7 @@ static NTSTATUS dcesrv_samr_CreateUser2(struct dcesrv_call_state *dce_call, TALL
 	/* Start a transaction, so we can query and do a subsequent atomic modify */
 	
 	/* create the user */
-	ret = samdb_add(d_state->sam_ctx, mem_ctx, msg);
+	ret = ldb_add(d_state->sam_ctx, msg);
 	switch (ret) {
 	case  LDB_SUCCESS:
 		break;
@@ -1586,7 +1586,7 @@ static NTSTATUS dcesrv_samr_CreateDomAlias(struct dcesrv_call_state *dce_call, T
 	samdb_msg_add_int(d_state->sam_ctx, mem_ctx, msg, "groupType", GTYPE_SECURITY_DOMAIN_LOCAL_GROUP);
 
 	/* create the alias */
-	ret = samdb_add(d_state->sam_ctx, mem_ctx, msg);
+	ret = ldb_add(d_state->sam_ctx, msg);
 	switch (ret) {
 	case LDB_SUCCESS:
 		break;
@@ -2263,7 +2263,7 @@ static NTSTATUS dcesrv_samr_DeleteDomainGroup(struct dcesrv_call_state *dce_call
 
 	a_state = h->data;
 
-	ret = samdb_delete(a_state->sam_ctx, mem_ctx, a_state->account_dn);
+	ret = ldb_delete(a_state->sam_ctx, a_state->account_dn);
 	if (ret != 0) {
 		return NT_STATUS_UNSUCCESSFUL;
 	}
@@ -2637,7 +2637,7 @@ static NTSTATUS dcesrv_samr_DeleteDomAlias(struct dcesrv_call_state *dce_call, T
 
 	a_state = h->data;
 
-	ret = samdb_delete(a_state->sam_ctx, mem_ctx, a_state->account_dn);
+	ret = ldb_delete(a_state->sam_ctx, a_state->account_dn);
 	if (ret != 0) {
 		return NT_STATUS_UNSUCCESSFUL;
 	}
@@ -2910,7 +2910,7 @@ static NTSTATUS dcesrv_samr_DeleteUser(struct dcesrv_call_state *dce_call, TALLO
 
 	a_state = h->data;
 
-	ret = samdb_delete(a_state->sam_ctx, mem_ctx, a_state->account_dn);
+	ret = ldb_delete(a_state->sam_ctx, a_state->account_dn);
 	if (ret != 0) {
 		return NT_STATUS_UNSUCCESSFUL;
 	}