svn commit: samba r17451 - in branches/SAMBA_3_0/source: groupdb
include passdb rpc_server utils
vlendec at samba.org
vlendec at samba.org
Tue Aug 8 08:26:42 GMT 2006
Author: vlendec
Date: 2006-08-08 08:26:40 +0000 (Tue, 08 Aug 2006)
New Revision: 17451
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=17451
Log:
Change pdb_getgrsid not to take a DOM_SID but a const DOM_SID * as an
argument.
Volker
Modified:
branches/SAMBA_3_0/source/groupdb/mapping.c
branches/SAMBA_3_0/source/include/passdb.h
branches/SAMBA_3_0/source/passdb/lookup_sid.c
branches/SAMBA_3_0/source/passdb/pdb_interface.c
branches/SAMBA_3_0/source/passdb/pdb_ldap.c
branches/SAMBA_3_0/source/rpc_server/srv_lsa_nt.c
branches/SAMBA_3_0/source/rpc_server/srv_samr_nt.c
branches/SAMBA_3_0/source/utils/net_groupmap.c
branches/SAMBA_3_0/source/utils/net_rpc_samsync.c
branches/SAMBA_3_0/source/utils/net_sam.c
Changeset:
Modified: branches/SAMBA_3_0/source/groupdb/mapping.c
===================================================================
--- branches/SAMBA_3_0/source/groupdb/mapping.c 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/groupdb/mapping.c 2006-08-08 08:26:40 UTC (rev 17451)
@@ -220,7 +220,7 @@
Return the sid and the type of the unix group.
****************************************************************************/
-static BOOL get_group_map_from_sid(DOM_SID sid, GROUP_MAP *map)
+static BOOL get_group_map_from_sid(const DOM_SID *sid, GROUP_MAP *map)
{
TDB_DATA kbuf, dbuf;
pstring key;
@@ -234,7 +234,7 @@
/* the key is the SID, retrieving is direct */
- sid_to_string(string_sid, &sid);
+ sid_to_string(string_sid, sid);
slprintf(key, sizeof(key), "%s%s", GROUP_PREFIX, string_sid);
kbuf.dptr = key;
@@ -254,7 +254,7 @@
return False;
}
- sid_copy(&map->sid, &sid);
+ sid_copy(&map->sid, sid);
return True;
}
@@ -588,7 +588,7 @@
return NT_STATUS_ACCESS_DENIED;
}
- if (!get_group_map_from_sid(*alias, &map))
+ if (!get_group_map_from_sid(alias, &map))
return NT_STATUS_NO_SUCH_ALIAS;
if ( (map.sid_name_use != SID_NAME_ALIAS) &&
@@ -691,7 +691,7 @@
return NT_STATUS_ACCESS_DENIED;
}
- if (!get_group_map_from_sid(*alias, &map))
+ if (!get_group_map_from_sid(alias, &map))
return NT_STATUS_NO_SUCH_ALIAS;
if ( (map.sid_name_use != SID_NAME_ALIAS) &&
@@ -796,7 +796,7 @@
/* get a domain group from it's SID */
-BOOL get_domain_group_from_sid(DOM_SID sid, GROUP_MAP *map)
+BOOL get_domain_group_from_sid(const DOM_SID *sid, GROUP_MAP *map)
{
struct group *grp;
BOOL ret;
@@ -819,12 +819,12 @@
if ( !ret ) {
uint32 rid;
- sid_peek_rid( &sid, &rid );
+ sid_peek_rid( sid, &rid );
if ( rid == DOMAIN_GROUP_RID_USERS ) {
fstrcpy( map->nt_name, "None" );
fstrcpy( map->comment, "Ordinary Users" );
- sid_copy( &map->sid, &sid );
+ sid_copy( &map->sid, sid );
map->sid_name_use = SID_NAME_DOM_GRP;
return True;
@@ -998,7 +998,7 @@
NTSTATUS pdb_default_getgrsid(struct pdb_methods *methods, GROUP_MAP *map,
- DOM_SID sid)
+ const DOM_SID *sid)
{
return get_group_map_from_sid(sid, map) ?
NT_STATUS_OK : NT_STATUS_UNSUCCESSFUL;
@@ -1138,7 +1138,7 @@
{
GROUP_MAP map;
- if (!pdb_getgrsid(&map, *sid))
+ if (!pdb_getgrsid(&map, sid))
return NT_STATUS_NO_SUCH_ALIAS;
if ((map.sid_name_use != SID_NAME_ALIAS) &&
@@ -1161,7 +1161,7 @@
{
GROUP_MAP map;
- if (!pdb_getgrsid(&map, *sid))
+ if (!pdb_getgrsid(&map, sid))
return NT_STATUS_NO_SUCH_ALIAS;
fstrcpy(map.nt_name, info->acct_name);
@@ -1285,7 +1285,7 @@
BOOL res;
become_root();
- res = get_domain_group_from_sid(*sid, &map);
+ res = get_domain_group_from_sid(sid, &map);
unbecome_root();
if (!res)
@@ -1301,7 +1301,7 @@
{
GROUP_MAP map;
- if (!get_domain_group_from_sid(*sid, &map))
+ if (!get_domain_group_from_sid(sid, &map))
return False;
fstrcpy(map.nt_name, info->acct_name);
Modified: branches/SAMBA_3_0/source/include/passdb.h
===================================================================
--- branches/SAMBA_3_0/source/include/passdb.h 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/include/passdb.h 2006-08-08 08:26:40 UTC (rev 17451)
@@ -244,7 +244,7 @@
* enum SID_NAME_USE rather than uint32.
*/
-#define PASSDB_INTERFACE_VERSION 14
+#define PASSDB_INTERFACE_VERSION 15
struct pdb_methods
{
@@ -277,7 +277,8 @@
NTSTATUS (*update_login_attempts)(struct pdb_methods *methods, struct samu *sam_acct, BOOL success);
- NTSTATUS (*getgrsid)(struct pdb_methods *methods, GROUP_MAP *map, DOM_SID sid);
+ NTSTATUS (*getgrsid)(struct pdb_methods *methods, GROUP_MAP *map,
+ const DOM_SID *sid);
NTSTATUS (*getgrgid)(struct pdb_methods *methods, GROUP_MAP *map, gid_t gid);
Modified: branches/SAMBA_3_0/source/passdb/lookup_sid.c
===================================================================
--- branches/SAMBA_3_0/source/passdb/lookup_sid.c 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/passdb/lookup_sid.c 2006-08-08 08:26:40 UTC (rev 17451)
@@ -1352,7 +1352,7 @@
if ((sid_check_is_in_builtin(psid) ||
sid_check_is_in_wellknown_domain(psid))) {
- if (pdb_getgrsid(&map, *psid)) {
+ if (pdb_getgrsid(&map, psid)) {
*pgid = map.gid;
goto done;
}
Modified: branches/SAMBA_3_0/source/passdb/pdb_interface.c
===================================================================
--- branches/SAMBA_3_0/source/passdb/pdb_interface.c 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/passdb/pdb_interface.c 2006-08-08 08:26:40 UTC (rev 17451)
@@ -561,7 +561,7 @@
return pdb->update_login_attempts(pdb, sam_acct, success);
}
-BOOL pdb_getgrsid(GROUP_MAP *map, DOM_SID sid)
+BOOL pdb_getgrsid(GROUP_MAP *map, const DOM_SID *sid)
{
struct pdb_methods *pdb = pdb_get_methods();
return NT_STATUS_IS_OK(pdb->getgrsid(pdb, map, sid));
@@ -636,7 +636,7 @@
sid_compose(&group_sid, get_global_sam_sid(), rid);
- if (!get_domain_group_from_sid(group_sid, &map)) {
+ if (!get_domain_group_from_sid(&group_sid, &map)) {
DEBUG(10, ("Could not find group for rid %d\n", rid));
return NT_STATUS_NO_SUCH_GROUP;
}
@@ -812,7 +812,7 @@
sid_compose(&group_sid, get_global_sam_sid(), group_rid);
sid_compose(&member_sid, get_global_sam_sid(), member_rid);
- if (!get_domain_group_from_sid(group_sid, &map) ||
+ if (!get_domain_group_from_sid(&group_sid, &map) ||
(map.gid == (gid_t)-1) ||
((grp = getgrgid(map.gid)) == NULL)) {
return NT_STATUS_NO_SUCH_GROUP;
@@ -874,7 +874,7 @@
sid_compose(&group_sid, get_global_sam_sid(), group_rid);
sid_compose(&member_sid, get_global_sam_sid(), member_rid);
- if (!get_domain_group_from_sid(group_sid, &map) ||
+ if (!get_domain_group_from_sid(&group_sid, &map) ||
(map.gid == (gid_t)-1) ||
((grp = getgrgid(map.gid)) == NULL)) {
return NT_STATUS_NO_SUCH_GROUP;
@@ -1276,7 +1276,7 @@
if (sid_peek_check_rid(&global_sid_Builtin, sid, &rid)) {
/* Here we only have aliases */
GROUP_MAP map;
- if (!NT_STATUS_IS_OK(methods->getgrsid(methods, &map, *sid))) {
+ if (!NT_STATUS_IS_OK(methods->getgrsid(methods, &map, sid))) {
DEBUG(10, ("Could not find map for sid %s\n",
sid_string_static(sid)));
goto done;
@@ -1522,7 +1522,7 @@
}
TALLOC_FREE(sam_account);
- ret = pdb_getgrsid(&map, sid);
+ ret = pdb_getgrsid(&map, &sid);
unbecome_root();
/* END BECOME_ROOT BLOCK */
Modified: branches/SAMBA_3_0/source/passdb/pdb_ldap.c
===================================================================
--- branches/SAMBA_3_0/source/passdb/pdb_ldap.c 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/passdb/pdb_ldap.c 2006-08-08 08:26:40 UTC (rev 17451)
@@ -2245,14 +2245,14 @@
*********************************************************************/
static NTSTATUS ldapsam_getgrsid(struct pdb_methods *methods, GROUP_MAP *map,
- DOM_SID sid)
+ const DOM_SID *sid)
{
pstring filter;
pstr_sprintf(filter, "(&(objectClass=%s)(%s=%s))",
LDAP_OBJ_GROUPMAP,
get_attr_key2string(groupmap_attr_list, LDAP_ATTR_GROUP_SID),
- sid_string_static(&sid));
+ sid_string_static(sid));
return ldapsam_getgroup(methods, filter, map);
}
Modified: branches/SAMBA_3_0/source/rpc_server/srv_lsa_nt.c
===================================================================
--- branches/SAMBA_3_0/source/rpc_server/srv_lsa_nt.c 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/rpc_server/srv_lsa_nt.c 2006-08-08 08:26:40 UTC (rev 17451)
@@ -1730,7 +1730,7 @@
if ( !nt_token_check_domain_rid( p->pipe_user.nt_user_token, DOMAIN_GROUP_RID_ADMINS ) )
return NT_STATUS_ACCESS_DENIED;
- if (!pdb_getgrsid(&map, info->sid))
+ if (!pdb_getgrsid(&map, &info->sid))
return NT_STATUS_NO_SUCH_GROUP;
return pdb_update_group_mapping_entry(&map);
Modified: branches/SAMBA_3_0/source/rpc_server/srv_samr_nt.c
===================================================================
--- branches/SAMBA_3_0/source/rpc_server/srv_samr_nt.c 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/rpc_server/srv_samr_nt.c 2006-08-08 08:26:40 UTC (rev 17451)
@@ -4438,7 +4438,7 @@
}
become_root();
- ret = get_domain_group_from_sid(group_sid, &map);
+ ret = get_domain_group_from_sid(&group_sid, &map);
unbecome_root();
if (!ret)
return NT_STATUS_INVALID_HANDLE;
@@ -4535,7 +4535,7 @@
}
become_root();
- result = get_domain_group_from_sid(group_sid, &map);
+ result = get_domain_group_from_sid(&group_sid, &map);
unbecome_root();
if (!result)
return NT_STATUS_NO_SUCH_GROUP;
@@ -4754,7 +4754,7 @@
/* check if that group really exists */
become_root();
- ret = get_domain_group_from_sid(info->sid, &map);
+ ret = get_domain_group_from_sid(&info->sid, &map);
unbecome_root();
if (!ret)
return NT_STATUS_NO_SUCH_GROUP;
Modified: branches/SAMBA_3_0/source/utils/net_groupmap.c
===================================================================
--- branches/SAMBA_3_0/source/utils/net_groupmap.c 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/utils/net_groupmap.c 2006-08-08 08:26:40 UTC (rev 17451)
@@ -153,7 +153,7 @@
}
/* Get the current mapping from the database */
- if(!pdb_getgrsid(&map, sid)) {
+ if(!pdb_getgrsid(&map, &sid)) {
d_fprintf(stderr, "Failure to local group SID in the database\n");
return -1;
}
@@ -404,7 +404,7 @@
}
/* Get the current mapping from the database */
- if(!pdb_getgrsid(&map, sid)) {
+ if(!pdb_getgrsid(&map, &sid)) {
d_fprintf(stderr, "Failure to local group SID in the database\n");
return -1;
}
@@ -539,7 +539,7 @@
DOM_SID sid;
have_map = ( (strncmp(ntgroup, "S-", 2) == 0) &&
string_to_sid(&sid, ntgroup) &&
- pdb_getgrsid(&map, sid) );
+ pdb_getgrsid(&map, &sid) );
}
if (!have_map) {
Modified: branches/SAMBA_3_0/source/utils/net_rpc_samsync.c
===================================================================
--- branches/SAMBA_3_0/source/utils/net_rpc_samsync.c 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/utils/net_rpc_samsync.c 2006-08-08 08:26:40 UTC (rev 17451)
@@ -588,7 +588,7 @@
group_sid = *pdb_get_group_sid(sam_account);
- if (!pdb_getgrsid(&map, group_sid)) {
+ if (!pdb_getgrsid(&map, &group_sid)) {
DEBUG(0, ("Primary group of %s has no mapping!\n",
pdb_get_username(sam_account)));
} else {
@@ -630,7 +630,7 @@
sid_append_rid(&group_sid, rid);
sid_to_string(sid_string, &group_sid);
- if (pdb_getgrsid(&map, group_sid)) {
+ if (pdb_getgrsid(&map, &group_sid)) {
if ( map.gid != -1 )
grp = getgrgid(map.gid);
insert = False;
@@ -689,7 +689,7 @@
sid_copy(&group_sid, get_global_sam_sid());
sid_append_rid(&group_sid, rid);
- if (!get_domain_group_from_sid(group_sid, &map)) {
+ if (!get_domain_group_from_sid(&group_sid, &map)) {
DEBUG(0, ("Could not find global group %d\n", rid));
return NT_STATUS_NO_SUCH_GROUP;
}
@@ -805,7 +805,6 @@
fstring comment;
struct group *grp = NULL;
DOM_SID alias_sid;
- fstring sid_string;
GROUP_MAP map;
BOOL insert = True;
@@ -815,9 +814,8 @@
/* Find out whether the group is already mapped */
sid_copy(&alias_sid, &dom_sid);
sid_append_rid(&alias_sid, rid);
- sid_to_string(sid_string, &alias_sid);
- if (pdb_getgrsid(&map, alias_sid)) {
+ if (pdb_getgrsid(&map, &alias_sid)) {
grp = getgrgid(map.gid);
insert = False;
}
Modified: branches/SAMBA_3_0/source/utils/net_sam.c
===================================================================
--- branches/SAMBA_3_0/source/utils/net_sam.c 2006-08-07 20:43:06 UTC (rev 17450)
+++ branches/SAMBA_3_0/source/utils/net_sam.c 2006-08-08 08:26:40 UTC (rev 17451)
@@ -330,7 +330,7 @@
return -1;
}
- if (!pdb_getgrsid(&map, sid)) {
+ if (!pdb_getgrsid(&map, &sid)) {
d_fprintf(stderr, "Could not load group %s\n", argv[0]);
return -1;
}
@@ -882,7 +882,7 @@
sid_compose(&gsid, get_global_sam_sid(), DOMAIN_GROUP_RID_USERS);
- if (!pdb_getgrsid(&gmap, gsid)) {
+ if (!pdb_getgrsid(&gmap, &gsid)) {
LDAPMod **mods = NULL;
char *dn;
char *uname;
@@ -935,7 +935,7 @@
sid_compose(&gsid, get_global_sam_sid(), DOMAIN_GROUP_RID_ADMINS);
- if (!pdb_getgrsid(&gmap, gsid)) {
+ if (!pdb_getgrsid(&gmap, &gsid)) {
LDAPMod **mods = NULL;
char *dn;
char *uname;
More information about the samba-cvs
mailing list