summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimo Sorce <idra@samba.org>2006-02-27 14:45:27 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 11:10:50 -0500
commitaec8de1703cd034a253b06b26e849c01d8ce131f (patch)
treedcdaf74864fe218fbdd4e74a3b05411b081bf756
parent1006c89e1b71b9b9d2c8453e07fd19b223e95415 (diff)
downloadsamba-aec8de1703cd034a253b06b26e849c01d8ce131f.tar.gz
samba-aec8de1703cd034a253b06b26e849c01d8ce131f.tar.bz2
samba-aec8de1703cd034a253b06b26e849c01d8ce131f.zip
r13715: Put back the code that actually modify the account,
removed, I presume by mistake, by Jerry in the recent patch the removes the primary group SID stuff. set_user_info_21 is called to update many other things like the description of a user for example (that's what failed on me). Jerry, please review this one. (This used to be commit 239a37d201168d095f600042b1ffcd047f18ba8a)
-rw-r--r--source3/rpc_server/srv_samr_nt.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/source3/rpc_server/srv_samr_nt.c b/source3/rpc_server/srv_samr_nt.c
index fa5b080634..de6c28a38d 100644
--- a/source3/rpc_server/srv_samr_nt.c
+++ b/source3/rpc_server/srv_samr_nt.c
@@ -3132,6 +3132,12 @@ static NTSTATUS set_user_info_21(TALLOC_CTX *mem_ctx, SAM_USER_INFO_21 *id21,
primary group SID is generated solely from the user's Unix
primary group. */
+ /* write the change out */
+ if(!NT_STATUS_IS_OK(status = pdb_update_sam_account(pwd))) {
+ TALLOC_FREE(pwd);
+ return status;
+ }
+
TALLOC_FREE(pwd);
return NT_STATUS_OK;