s3:passdb: fix a type Domain Users has RID -513

[samba.git] / source3 / passdb / pdb_get_set.c

diff --git a/source3/passdb/pdb_get_set.c b/source3/passdb/pdb_get_set.c
index 7fc9f92b2c2a46068df31d4712412a0c035812c6..555b20730c22ba7da566304ef5fff62f8d33e4c5 100644 (file)
--- a/source3/passdb/pdb_get_set.c
+++ b/source3/passdb/pdb_get_set.c
@@ -236,11 +236,10 @@ const DOM_SID *pdb_get_group_sid(struct samu *sampass)
                        pwd->pw_name, sid_type_lookup(type)));
        }
 
-       /* Just set it to the 'Domain Users' RID of 512 which will 
+       /* Just set it to the 'Domain Users' RID of 513 which will
           always resolve to a name */
 
-       sid_copy( gsid, get_global_sam_sid() );
-       sid_append_rid( gsid, DOMAIN_GROUP_RID_USERS );
+       sid_compose(gsid, get_global_sam_sid(), DOMAIN_GROUP_RID_USERS);
 
        sampass->group_sid = gsid;
 
@@ -552,8 +551,8 @@ bool pdb_set_group_sid(struct samu *sampass, const DOM_SID *g_sid, enum pdb_valu
        if ( sid_to_gid( g_sid, &gid ) ) {
                sid_copy(sampass->group_sid, g_sid);
        } else {
-               sid_copy( sampass->group_sid, get_global_sam_sid() );
-               sid_append_rid( sampass->group_sid, DOMAIN_GROUP_RID_USERS );
+               sid_compose(sampass->group_sid, get_global_sam_sid(),
+                           DOMAIN_GROUP_RID_USERS);
        }
 
        DEBUG(10, ("pdb_set_group_sid: setting group sid %s\n",