diff options
| author | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-12-02 12:23:53 +0100 |
|---|---|---|
| committer | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-12-02 17:17:55 +0100 |
| commit | 7a5e47bf4e55d7b53f487ed5eb6eab31e20bdcc5 (patch) | |
| tree | 95c3a73d467f01e8f7196f81a17d7ffbd1c6836f | |
| parent | 6f42da795e5cce29c122aea1a6818c1ae1d99ae6 (diff) | |
| download | samba-7a5e47bf4e55d7b53f487ed5eb6eab31e20bdcc5.tar.gz samba-7a5e47bf4e55d7b53f487ed5eb6eab31e20bdcc5.tar.bz2 samba-7a5e47bf4e55d7b53f487ed5eb6eab31e20bdcc5.zip | |
s4:auth/sam.c-"authsam_expand_nested_groups" - don't fail if we've memberships on non-SAM objects
This can be expected (think at a membership of a "groupOfNames" group) and we
shouldn't blame about it.
This fixes a bug reported on the technical mailing list.
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Thu Dec 2 17:17:56 CET 2010 on sn-devel-104
| -rw-r--r-- | source4/auth/sam.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/source4/auth/sam.c b/source4/auth/sam.c index b98830a0a9..6203b960c4 100644 --- a/source4/auth/sam.c +++ b/source4/auth/sam.c @@ -326,11 +326,11 @@ NTSTATUS authsam_expand_nested_groups(struct ldb_context *sam_ctx, status = dsdb_get_extended_dn_sid(dn, &sid, "SID"); if (!NT_STATUS_IS_OK(status)) { - DEBUG(0, (__location__ ": when parsing DN %s we failed to find our SID component, so we cannot calculate the group token: %s\n", - ldb_dn_get_extended_linearized(tmp_ctx, dn, 1), - nt_errstr(status))); + /* If we fail finding a SID then this is no error since it could + * be a non SAM object - e.g. a group with object class + * "groupOfNames" */ talloc_free(tmp_ctx); - return NT_STATUS_INTERNAL_DB_CORRUPTION; + return NT_STATUS_OK; } if (!sam_ctx) { |