svn commit: samba r13715 - branches/SAMBA_3_0/source/rpc_server trunk/source/rpc_server

[idra at samba.org]

Author: idra
Date: 2006-02-27 14:45:27 +0000 (Mon, 27 Feb 2006)
New Revision: 13715

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=13715

Log:


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.


Modified:
   branches/SAMBA_3_0/source/rpc_server/srv_samr_nt.c
   trunk/source/rpc_server/srv_samr_nt.c


Changeset:
Modified: branches/SAMBA_3_0/source/rpc_server/srv_samr_nt.c
===================================================================
--- branches/SAMBA_3_0/source/rpc_server/srv_samr_nt.c	2006-02-27 14:18:52 UTC (rev 13714)
+++ branches/SAMBA_3_0/source/rpc_server/srv_samr_nt.c	2006-02-27 14:45:27 UTC (rev 13715)
@@ -3132,6 +3132,12 @@
 	   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;

Modified: trunk/source/rpc_server/srv_samr_nt.c
===================================================================
--- trunk/source/rpc_server/srv_samr_nt.c	2006-02-27 14:18:52 UTC (rev 13714)
+++ trunk/source/rpc_server/srv_samr_nt.c	2006-02-27 14:45:27 UTC (rev 13715)
@@ -3132,6 +3132,12 @@
 	   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;