svn commit: samba r6274 - in trunk/source: nsswitch sam

vlendec at samba.org vlendec at samba.org
Sun Apr 10 14:21:01 GMT 2005


Author: vlendec
Date: 2005-04-10 14:21:00 +0000 (Sun, 10 Apr 2005)
New Revision: 6274

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

Log:
Remove some unused code, minor cleanup
Modified:
   trunk/source/nsswitch/winbindd_util.c
   trunk/source/sam/idmap_util.c


Changeset:
Modified: trunk/source/nsswitch/winbindd_util.c
===================================================================
--- trunk/source/nsswitch/winbindd_util.c	2005-04-10 14:20:29 UTC (rev 6273)
+++ trunk/source/nsswitch/winbindd_util.c	2005-04-10 14:21:00 UTC (rev 6274)
@@ -87,13 +87,7 @@
 	if (sid == NULL)
 		return False;
 
-	if (sid_compare_domain(sid, get_global_sam_sid()) == 0)
-		return True;
-
-	if (sid_compare_domain(sid, &global_sid_Builtin) == 0)
-		return True;
-
-	return False;
+	return (sid_check_is_domain(sid) || sid_check_is_builtin(sid));
 }
 
 

Modified: trunk/source/sam/idmap_util.c
===================================================================
--- trunk/source/sam/idmap_util.c	2005-04-10 14:20:29 UTC (rev 6273)
+++ trunk/source/sam/idmap_util.c	2005-04-10 14:21:00 UTC (rev 6274)
@@ -184,106 +184,3 @@
 
 	return ret;
 }
-
-
-/***************************************************************************
- Check first, call set_mapping if it doesn't already exist.
-***************************************************************************/
-
-static NTSTATUS wellknown_id_init(DOM_SID *sid, unid_t id, int flags)
-{
-	unid_t storedid;
-	int qflags = flags | ID_QUERY_ONLY;
-
-	if (!NT_STATUS_IS_OK(idmap_get_id_from_sid(&storedid, &qflags, sid))) {
-		return idmap_set_mapping(sid, id, flags);
-	} else {
-		if (flags == ID_USERID && id.uid != storedid.uid) {
-			DEBUG(0,("wellknown_id_init: WARNING ! Stored uid %u for SID %s is not the same as the requested uid %u\n",
-				(unsigned int)storedid.uid, sid_string_static(sid), (unsigned int)id.uid ));
-			DEBUG(0,("wellknown_id_init: Attempting to overwrite old mapping with new.\n"));
-			return idmap_set_mapping(sid, id, flags);
-		} else if (flags == ID_GROUPID && id.gid != storedid.gid) {
-			DEBUG(0,("wellknown_id_init: WARNING ! Stored gid %u for SID %s is not the same as the requested gid %u\n",
-				(unsigned int)storedid.gid, sid_string_static(sid), (unsigned int)id.gid ));
-			DEBUG(0,("wellknown_id_init: Attempting to overwrite old mapping with new.\n"));
-			return idmap_set_mapping(sid, id, flags);
-		}
-	}
-	return NT_STATUS_OK;
-}
-
-/***************************************************************************
- Initialize idmap withWellknown SIDs like Guest, that are necessary
- to make samba run properly.
-***************************************************************************/
-
-BOOL idmap_init_wellknown_sids(void)
-{
-	const char *guest_account = lp_guestaccount();
-	struct passwd *pass;
-	GROUP_MAP *map=NULL;
-	int num_entries=0;
-	DOM_SID sid;
-	unid_t id;
-	fstring sid_string;
-
-	if (!(guest_account && *guest_account)) {
-		DEBUG(1, ("NULL guest account!?!?\n"));
-		return False;
-	}
-
-	pass = getpwnam_alloc(guest_account);
-	if (!pass) {
-		return False;
-	}
-
-	/* Fill in the SID for the guest account. */
-	id.uid = pass->pw_uid;
-	sid_copy(&sid, get_global_sam_sid());
-	sid_append_rid(&sid, DOMAIN_USER_RID_GUEST);
-
-	if (!NT_STATUS_IS_OK(wellknown_id_init(&sid, id, ID_USERID))) {
-		DEBUG(0, ("Failed to setup UID mapping for GUEST (%s) to (%u)\n", 
-			  sid_to_string(sid_string, &sid), (unsigned int)id.uid));
-		passwd_free(&pass);
-		return False;
-	}
-
-	/* check if DOMAIN_GROUP_RID_GUESTS SID is set, if not store the
-	 * guest account gid as mapping */
-	id.gid = pass->pw_gid;
-	sid_copy(&sid, get_global_sam_sid());
-	sid_append_rid(&sid, DOMAIN_GROUP_RID_GUESTS);
-	if (!NT_STATUS_IS_OK(wellknown_id_init(&sid, id, ID_GROUPID))) {
-		DEBUG(0, ("Failed to setup GID mapping for Group DOMAIN GUESTS (%s) to (%u)\n", 
-			  sid_to_string(sid_string, &sid), (unsigned int)id.gid));
-		passwd_free(&pass);
-		return False;
-	}
-
-	passwd_free(&pass);
-	/* now fill in group mappings */
-	if(pdb_enum_group_mapping(SID_NAME_UNKNOWN, &map, &num_entries, ENUM_ONLY_MAPPED)) {
-		int i;
-
-		for (i = 0; i < num_entries; i++) {
-			id.gid = map[i].gid;
-			wellknown_id_init(&map[i].sid, id, ID_GROUPID);
-		}
-		SAFE_FREE(map);
-	}
-
-	/* Fill in the SID for the administrator account. */
-	id.uid = 0;
-	sid_copy(&sid, get_global_sam_sid());
-	sid_append_rid(&sid, DOMAIN_USER_RID_ADMIN);
-
-	if (!NT_STATUS_IS_OK(wellknown_id_init(&sid, id, ID_USERID))) {
-		DEBUG(0, ("Failed to setup UID mapping for ADMINISTRATOR (%s) to (%u)\n", 
-			  sid_to_string(sid_string, &sid), (unsigned int)id.uid));
-		return False;
-	}
-
-	return True;
-}



More information about the samba-cvs mailing list