[SCM] Samba Shared Repository - branch master updated - release-4-0-0alpha7-1510-gf47b220

Jeremy Allison jra at samba.org
Fri May 8 17:14:13 GMT 2009


The branch, master has been updated
       via  f47b220ce1bd8632205679bbba8f679f4470360b (commit)
      from  b79eff843be392f3065e912edca1434081d93c44 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit f47b220ce1bd8632205679bbba8f679f4470360b
Author: Jeremy Allison <jra at samba.org>
Date:   Fri May 8 10:11:38 2009 -0700

    Expand the comment explaining why user_in_group_sid is
    not reliable for winbindd users from foreign domains.
    Jeremy.

-----------------------------------------------------------------------

Summary of changes:
 source3/smbd/posix_acls.c |    7 ++++++-
 1 files changed, 6 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/posix_acls.c b/source3/smbd/posix_acls.c
index abfb00c..8d172e1 100644
--- a/source3/smbd/posix_acls.c
+++ b/source3/smbd/posix_acls.c
@@ -1297,7 +1297,12 @@ static bool uid_entry_in_group( canon_ace *uid_ace, canon_ace *group_ace )
 		return False;
 	}
 
-	/* notice that this is not reliable for users exported by winbindd! */
+	/*
+	 * user_in_group_sid() uses create_token_from_username()
+	 * which creates an artificial NT token given just a username,
+	 * so this is not reliable for users from foreign domains
+	 * exported by winbindd!
+	 */
 	return user_in_group_sid(u_name, &group_ace->trustee);
 }
 


-- 
Samba Shared Repository


More information about the samba-cvs mailing list