mirror of
https://github.com/samba-team/samba.git
synced 2025-02-02 09:47:23 +03:00
s4:dcesrv_samr_EnumDomainGroups/Aliases - when we don't get a SID then the database is corrupted
Group/User/Alias entries do always have a SID (it's a mandatory attribute in the SAM directory)!
This commit is contained in:
parent
4659b3c4fd
commit
4a8ee9a333
@ -1112,8 +1112,9 @@ static NTSTATUS dcesrv_samr_EnumDomainGroups(struct dcesrv_call_state *dce_call,
|
||||
|
||||
group_sid = samdb_result_dom_sid(mem_ctx, res[i],
|
||||
"objectSid");
|
||||
if (group_sid == NULL)
|
||||
continue;
|
||||
if (group_sid == NULL) {
|
||||
return NT_STATUS_INTERNAL_DB_CORRUPTION;
|
||||
}
|
||||
|
||||
entries[count].idx =
|
||||
group_sid->sub_auths[group_sid->num_auths-1];
|
||||
@ -1467,8 +1468,9 @@ static NTSTATUS dcesrv_samr_EnumDomainAliases(struct dcesrv_call_state *dce_call
|
||||
alias_sid = samdb_result_dom_sid(mem_ctx, res[i],
|
||||
"objectSid");
|
||||
|
||||
if (alias_sid == NULL)
|
||||
continue;
|
||||
if (alias_sid == NULL) {
|
||||
return NT_STATUS_INTERNAL_DB_CORRUPTION;
|
||||
}
|
||||
|
||||
entries[count].idx =
|
||||
alias_sid->sub_auths[alias_sid->num_auths-1];
|
||||
|
Loading…
x
Reference in New Issue
Block a user