From aec8de1703cd034a253b06b26e849c01d8ce131f Mon Sep 17 00:00:00 2001 From: Simo Sorce Date: Mon, 27 Feb 2006 14:45:27 +0000 Subject: 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) --- source3/rpc_server/srv_samr_nt.c | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'source3/rpc_server') 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; -- cgit