summaryrefslogtreecommitdiff
path: root/source4/rpc_server
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>2004-11-29 06:19:50 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 13:06:12 -0500
commitf11aa2329666768fc219292be3b51a646aba870a (patch)
tree1bd4f4485c5aafc6008c84623bd1427b7856469b /source4/rpc_server
parent96f4d9815346edbb277583d417b4a8ae6c1c0a09 (diff)
downloadsamba-f11aa2329666768fc219292be3b51a646aba870a.tar.gz
samba-f11aa2329666768fc219292be3b51a646aba870a.tar.bz2
samba-f11aa2329666768fc219292be3b51a646aba870a.zip
r3994: - removed the unused reference count code in lsa server
- fixed the sid_index field in lsa LookupSids and LookupNames (This used to be commit 677f701e71609d82376b1ea2fa9ebc3521896671)
Diffstat (limited to 'source4/rpc_server')
-rw-r--r--source4/rpc_server/lsa/dcesrv_lsa.c76
1 files changed, 39 insertions, 37 deletions
diff --git a/source4/rpc_server/lsa/dcesrv_lsa.c b/source4/rpc_server/lsa/dcesrv_lsa.c
index 4cebc3f5aa..652a8edf6a 100644
--- a/source4/rpc_server/lsa/dcesrv_lsa.c
+++ b/source4/rpc_server/lsa/dcesrv_lsa.c
@@ -40,34 +40,23 @@ enum lsa_handle {
state associated with a lsa_OpenPolicy() operation
*/
struct lsa_policy_state {
- int reference_count;
void *sam_ctx;
struct sidmap_context *sidmap;
uint32_t access_mask;
const char *domain_dn;
const char *domain_name;
struct dom_sid *domain_sid;
+ struct dom_sid *builtin_sid;
};
/*
- destroy policy state
-*/
-static void lsa_Policy_close(struct lsa_policy_state *state)
-{
- state->reference_count--;
- if (state->reference_count == 0) {
- talloc_free(state);
- }
-}
-
-/*
destroy an open policy. This closes the database connection
*/
static void lsa_Policy_destroy(struct dcesrv_connection *conn, struct dcesrv_handle *h)
{
struct lsa_policy_state *state = h->data;
- lsa_Policy_close(state);
+ talloc_free(state);
}
/*
@@ -195,6 +184,12 @@ static NTSTATUS lsa_OpenPolicy2(struct dcesrv_call_state *dce_call, TALLOC_CTX *
return NT_STATUS_NO_SUCH_DOMAIN;
}
+ state->builtin_sid = dom_sid_parse_talloc(state, SID_BUILTIN);
+ if (!state->builtin_sid) {
+ talloc_free(state);
+ return NT_STATUS_NO_SUCH_DOMAIN;
+ }
+
state->domain_name = samdb_search_string(state->sam_ctx, state, NULL,
"name", "dn=%s", state->domain_dn);
if (!state->domain_name) {
@@ -212,7 +207,6 @@ static NTSTATUS lsa_OpenPolicy2(struct dcesrv_call_state *dce_call, TALLOC_CTX *
handle->data = state;
handle->destroy = lsa_Policy_destroy;
- state->reference_count = 1;
state->access_mask = r->in.access_mask;
*r->out.handle = handle->wire_handle;
@@ -417,6 +411,12 @@ static NTSTATUS lsa_authority_name(struct lsa_policy_state *state,
return NT_STATUS_OK;
}
+ if (dom_sid_in_domain(state->builtin_sid, sid)) {
+ *authority_name = "BUILTIN";
+ *authority_sid = state->builtin_sid;
+ return NT_STATUS_OK;
+ }
+
*authority_sid = dom_sid_dup(mem_ctx, sid);
if (*authority_sid == NULL) {
return NT_STATUS_NO_MEMORY;
@@ -435,7 +435,8 @@ static NTSTATUS lsa_authority_name(struct lsa_policy_state *state,
*/
static NTSTATUS lsa_authority_list(struct lsa_policy_state *state, TALLOC_CTX *mem_ctx,
struct dom_sid *sid,
- struct lsa_RefDomainList *domains)
+ struct lsa_RefDomainList *domains,
+ uint32_t *sid_index)
{
NTSTATUS status;
const char *authority_name;
@@ -452,21 +453,22 @@ static NTSTATUS lsa_authority_list(struct lsa_policy_state *state, TALLOC_CTX *m
/* see if we've already done this authority name */
for (i=0;i<domains->count;i++) {
if (strcmp(authority_name, domains->domains[i].name.string) == 0) {
- break;
+ *sid_index = i;
+ return NT_STATUS_OK;
}
}
- if (i == domains->count) {
- domains->domains = talloc_realloc_p(domains,
- domains->domains,
- struct lsa_TrustInformation,
- domains->count+1);
- if (domains->domains == NULL) {
- return NT_STATUS_NO_MEMORY;
- }
- domains->domains[i].name.string = authority_name;
- domains->domains[i].sid = authority_sid;
- domains->count++;
+
+ domains->domains = talloc_realloc_p(domains,
+ domains->domains,
+ struct lsa_TrustInformation,
+ domains->count+1);
+ if (domains->domains == NULL) {
+ return NT_STATUS_NO_MEMORY;
}
+ domains->domains[i].name.string = authority_name;
+ domains->domains[i].sid = authority_sid;
+ domains->count++;
+ *sid_index = i;
return NT_STATUS_OK;
}
@@ -542,7 +544,7 @@ static NTSTATUS lsa_LookupSids2(struct dcesrv_call_state *dce_call,
struct dom_sid *sid = r->in.sids->sids[i].sid;
char *sid_str = dom_sid_string(mem_ctx, sid);
const char *name;
- uint32_t atype, rtype;
+ uint32_t atype, rtype, sid_index;
NTSTATUS status2;
r->out.names->count++;
@@ -560,7 +562,7 @@ static NTSTATUS lsa_LookupSids2(struct dcesrv_call_state *dce_call,
}
/* work out the authority name */
- status2 = lsa_authority_list(state, mem_ctx, sid, r->out.domains);
+ status2 = lsa_authority_list(state, mem_ctx, sid, r->out.domains, &sid_index);
if (!NT_STATUS_IS_OK(status2)) {
return status2;
}
@@ -580,7 +582,7 @@ static NTSTATUS lsa_LookupSids2(struct dcesrv_call_state *dce_call,
r->out.names->names[i].sid_type = rtype;
r->out.names->names[i].name.string = name;
- r->out.names->names[i].sid_index = 0;
+ r->out.names->names[i].sid_index = sid_index;
r->out.names->names[i].unknown = 0;
}
@@ -1065,7 +1067,7 @@ static NTSTATUS lsa_LookupNames2(struct dcesrv_call_state *dce_call,
const char *sid_str;
struct ldb_message **res;
struct dom_sid *sid;
- uint32_t atype, rtype;
+ uint32_t atype, rtype, sid_index;
NTSTATUS status2;
r->out.sids->count++;
@@ -1106,15 +1108,15 @@ static NTSTATUS lsa_LookupNames2(struct dcesrv_call_state *dce_call,
continue;
}
- r->out.sids->sids[i].sid_type = rtype;
- r->out.sids->sids[i].rid = sid->sub_auths[sid->num_auths-1];
- r->out.sids->sids[i].sid_index = 0;
- r->out.sids->sids[i].unknown = 0;
-
- status2 = lsa_authority_list(state, mem_ctx, sid, r->out.domains);
+ status2 = lsa_authority_list(state, mem_ctx, sid, r->out.domains, &sid_index);
if (!NT_STATUS_IS_OK(status2)) {
return status2;
}
+
+ r->out.sids->sids[i].sid_type = rtype;
+ r->out.sids->sids[i].rid = sid->sub_auths[sid->num_auths-1];
+ r->out.sids->sids[i].sid_index = sid_index;
+ r->out.sids->sids[i].unknown = 0;
}
return status;