svn commit: samba r13541 - branches/SAMBA_3_0/source/auth trunk/source/auth

[jerry at samba.org]

Author: jerry
Date: 2006-02-17 13:30:34 +0000 (Fri, 17 Feb 2006)
New Revision: 13541

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

Log:
we have to wrap pen_enum_group_memberships() in become/unbecome_root()
blocks.  This fixes the problem I had with missing groups in the 
net_samlogon() reply from a Samba PDC.


Modified:
   branches/SAMBA_3_0/source/auth/auth_sam.c
   trunk/source/auth/auth_sam.c


Changeset:
Modified: branches/SAMBA_3_0/source/auth/auth_sam.c
===================================================================
--- branches/SAMBA_3_0/source/auth/auth_sam.c	2006-02-17 04:22:34 UTC (rev 13540)
+++ branches/SAMBA_3_0/source/auth/auth_sam.c	2006-02-17 13:30:34 UTC (rev 13541)
@@ -328,7 +328,11 @@
 		return nt_status;
 	}
 
-	if (!NT_STATUS_IS_OK(nt_status = make_server_info_sam(server_info, sampass))) {		
+	become_root();
+	nt_status = make_server_info_sam(server_info, sampass);
+	unbecome_root();
+
+	if (!NT_STATUS_IS_OK(nt_status)) {
 		DEBUG(0,("check_sam_security: make_server_info_sam() failed with '%s'\n", nt_errstr(nt_status)));
 		pdb_free_sam(&sampass);
 		data_blob_free(&user_sess_key);

Modified: trunk/source/auth/auth_sam.c
===================================================================
--- trunk/source/auth/auth_sam.c	2006-02-17 04:22:34 UTC (rev 13540)
+++ trunk/source/auth/auth_sam.c	2006-02-17 13:30:34 UTC (rev 13541)
@@ -328,7 +328,11 @@
 		return nt_status;
 	}
 
-	if (!NT_STATUS_IS_OK(nt_status = make_server_info_sam(server_info, sampass))) {		
+	become_root();
+	nt_status = make_server_info_sam(server_info, sampass);
+	unbecome_root();
+
+	if (!NT_STATUS_IS_OK(nt_status)) {
 		DEBUG(0,("check_sam_security: make_server_info_sam() failed with '%s'\n", nt_errstr(nt_status)));
 		pdb_free_sam(&sampass);
 		data_blob_free(&user_sess_key);