smbd: setting current_user stuff here is redundant

This is already handled by set_sec_ctx() below, we just have to pass in the
values instead of setting it here in this function before calling set_sec_ctx().

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>

diff --git a/source3/smbd/uid.c b/source3/smbd/uid.c
index 9b983df80db5151039d69bb53a7b0e9ae5603d35..f425f209bb95d77d03246df1f3c24de6ca6ee00b 100644 (file)
--- a/source3/smbd/uid.c
+++ b/source3/smbd/uid.c
@@ -413,14 +413,10 @@ static bool change_to_user_impersonate(connection_struct *conn,
                }
        }
 
-       /*Set current_user since we will immediately also call set_sec_ctx() */
-       current_user.ut.ngroups = num_groups;
-       current_user.ut.groups  = group_list;
-
        set_sec_ctx(uid,
                    gid,
-                   current_user.ut.ngroups,
-                   current_user.ut.groups,
+                   num_groups,
+                   group_list,
                    conn->session_info->security_token);
 
        current_user.conn = conn;