diff options
author | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-06-11 23:39:46 +0200 |
---|---|---|
committer | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-06-12 16:45:46 +0200 |
commit | 84bda98066e23b7963a6280155eafa88312724a9 (patch) | |
tree | 2d3b09557bac9ed872a3fa1620baf292db0a51f5 /source4/rpc_server/samr/dcesrv_samr.c | |
parent | bbb0b31911c41091dd79da3612b0227c68139da7 (diff) | |
download | samba-84bda98066e23b7963a6280155eafa88312724a9.tar.gz samba-84bda98066e23b7963a6280155eafa88312724a9.tar.bz2 samba-84bda98066e23b7963a6280155eafa88312724a9.zip |
s4:dcesrv_samr_EnumDomainUsers - make this call look more similar to "EnumDomainGroups" and "EnumDomainAliases"
That means that the lookup is now also done by "samdb_search_domain" to be more
consistent.
Diffstat (limited to 'source4/rpc_server/samr/dcesrv_samr.c')
-rw-r--r-- | source4/rpc_server/samr/dcesrv_samr.c | 51 |
1 files changed, 28 insertions, 23 deletions
diff --git a/source4/rpc_server/samr/dcesrv_samr.c b/source4/rpc_server/samr/dcesrv_samr.c index 2ab5155c10..9cb5113480 100644 --- a/source4/rpc_server/samr/dcesrv_samr.c +++ b/source4/rpc_server/samr/dcesrv_samr.c @@ -1259,10 +1259,9 @@ static NTSTATUS dcesrv_samr_EnumDomainUsers(struct dcesrv_call_state *dce_call, { struct dcesrv_handle *h; struct samr_domain_state *d_state; - struct ldb_result *res; - int ret; - unsigned int i; - uint32_t num_filtered_entries, first; + struct ldb_message **res; + int i, ldb_cnt; + uint32_t first, count; struct samr_SamEntry *entries; const char * const attrs[] = { "objectSid", "sAMAccountName", "userAccountControl", NULL }; @@ -1276,44 +1275,50 @@ static NTSTATUS dcesrv_samr_EnumDomainUsers(struct dcesrv_call_state *dce_call, d_state = h->data; - /* don't have to worry about users in the builtin domain, as there are none */ - ret = ldb_search(d_state->sam_ctx, mem_ctx, &res, d_state->domain_dn, LDB_SCOPE_SUBTREE, attrs, "objectClass=user"); - - if (ret != LDB_SUCCESS) { - DEBUG(3, ("Failed to search for Domain Users in %s: %s\n", - ldb_dn_get_linearized(d_state->domain_dn), ldb_errstring(d_state->sam_ctx))); + /* search for all domain users in this domain. This could possibly be + cached and resumed on resume_key */ + ldb_cnt = samdb_search_domain(d_state->sam_ctx, mem_ctx, + d_state->domain_dn, + &res, attrs, + d_state->domain_sid, + "(objectClass=user)"); + if (ldb_cnt < 0) { return NT_STATUS_INTERNAL_DB_CORRUPTION; } /* convert to SamEntry format */ - entries = talloc_array(mem_ctx, struct samr_SamEntry, res->count); + entries = talloc_array(mem_ctx, struct samr_SamEntry, ldb_cnt); if (!entries) { return NT_STATUS_NO_MEMORY; } - num_filtered_entries = 0; - for (i=0;i<res->count;i++) { + + count = 0; + + for (i=0;i<ldb_cnt;i++) { /* Check if a mask has been requested */ if (r->in.acct_flags - && ((samdb_result_acct_flags(d_state->sam_ctx, mem_ctx, res->msgs[i], - d_state->domain_dn) & r->in.acct_flags) == 0)) { + && ((samdb_result_acct_flags(d_state->sam_ctx, mem_ctx, + res[i], d_state->domain_dn) & r->in.acct_flags) == 0)) { continue; } - entries[num_filtered_entries].idx = samdb_result_rid_from_sid(mem_ctx, res->msgs[i], "objectSid", 0); - entries[num_filtered_entries].name.string = samdb_result_string(res->msgs[i], "sAMAccountName", ""); - num_filtered_entries++; + entries[count].idx = samdb_result_rid_from_sid(mem_ctx, res[i], + "objectSid", 0); + entries[count].name.string = samdb_result_string(res[i], + "sAMAccountName", ""); + count += 1; } /* sort the results by rid */ - TYPESAFE_QSORT(entries, num_filtered_entries, compare_SamEntry); + TYPESAFE_QSORT(entries, count, compare_SamEntry); /* find the first entry to return */ for (first=0; - first<num_filtered_entries && entries[first].idx <= *r->in.resume_handle; + first<count && entries[first].idx <= *r->in.resume_handle; first++) ; /* return the rest, limit by max_size. Note that we use the w2k3 element size value of 54 */ - *r->out.num_entries = num_filtered_entries - first; + *r->out.num_entries = count - first; *r->out.num_entries = MIN(*r->out.num_entries, 1+(r->in.max_size/SAMR_ENUM_USERS_MULTIPLIER)); @@ -1327,11 +1332,11 @@ static NTSTATUS dcesrv_samr_EnumDomainUsers(struct dcesrv_call_state *dce_call, *r->out.sam = sam; - if (first == num_filtered_entries) { + if (first == count) { return NT_STATUS_OK; } - if (*r->out.num_entries < num_filtered_entries - first) { + if (*r->out.num_entries < count - first) { *r->out.resume_handle = entries[first+*r->out.num_entries-1].idx; return STATUS_MORE_ENTRIES; } |