diff options
author | Jeremy Allison <jra@samba.org> | 2016-05-26 16:42:42 -0700 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2016-06-01 12:35:23 +0200 |
commit | 0ab3ef3aae8431790173eb41999239ad6b10af9c (patch) | |
tree | 4551819fe4369ff2b32823871c97c1876429f23a | |
parent | 615516bcabb08b0c4947b3fe030439c41f62d9bf (diff) | |
download | samba-0ab3ef3aae8431790173eb41999239ad6b10af9c.tar.gz |
s3: auth: Move the declaration of struct dom_sid tmp_sid to function level scope.
It's referred to outside of the {} brace scope it was defined in by
the following code:
uid_to_unix_users_sid(*uid, &tmp_sid);
user_sid = &tmp_sid;
As tmp_sid was going out of scope, user_sid was
being incorrectly set in the token sid list.
I think this *may* be the root cause of:
BUG: https://bugzilla.samba.org/show_bug.cgi?id=10618
But even if not this is an obvious error that must
be fixed.
Back-port from master: 1b3b89345480d16222da00753f973e36e2e0f92d
Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Volker Lendecke <vl@samba.org>
Autobuild-User(v4-2-test): Karolin Seeger <kseeger@samba.org>
Autobuild-Date(v4-2-test): Wed Jun 1 12:35:23 CEST 2016 on sn-devel-104
-rw-r--r-- | source3/auth/token_util.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/auth/token_util.c b/source3/auth/token_util.c index 9bb014c32cd..f6ce1315c08 100644 --- a/source3/auth/token_util.c +++ b/source3/auth/token_util.c @@ -673,6 +673,7 @@ static NTSTATUS create_token_from_sid(TALLOC_CTX *mem_ctx, gid_t *gids; struct dom_sid *group_sids; struct dom_sid unix_group_sid; + struct dom_sid tmp_sid; uint32_t num_group_sids; uint32_t num_gids; uint32_t i; @@ -756,7 +757,6 @@ static NTSTATUS create_token_from_sid(TALLOC_CTX *mem_ctx, *uid = sam_acct->unix_pw->pw_uid; } else if (sid_check_is_in_unix_users(user_sid)) { - struct dom_sid tmp_sid; uint32_t getgroups_num_group_sids; /* This is a unix user not in passdb. We need to ask nss * directly, without consulting passdb */ |