s3-auth Change type of num_sids to uint32_t
[samba.git] / source3 / lib / util_sid.c
index 222b32ed3a141f92de878e7695da8e97a45c2978..459636d5fcc94631f58d3549e2dcb445c2850d9c 100644 (file)
@@ -7,22 +7,25 @@
    Copyright (C) Stefan (metze) Metzmacher     2002
    Copyright (C) Simo Sorce                    2002
    Copyright (C) Jim McDonough <jmcd@us.ibm.com> 2005
-      
+
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
-   
+
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU General Public License for more details.
-   
+
    You should have received a copy of the GNU General Public License
    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "includes.h"
+#include "../librpc/gen_ndr/ndr_security.h"
+#include "../librpc/gen_ndr/netlogon.h"
+#include "../libcli/security/dom_sid.h"
 
 /*
  * Some useful sids, more well known sids can be found at
  */
 
 
-const DOM_SID global_sid_World_Domain =               /* Everyone domain */
+const struct dom_sid global_sid_World_Domain =               /* Everyone domain */
 { 1, 0, {0,0,0,0,0,1}, {0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_World =                      /* Everyone */
+const struct dom_sid global_sid_World =                      /* Everyone */
 { 1, 1, {0,0,0,0,0,1}, {0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Creator_Owner_Domain =       /* Creator Owner domain */
+const struct dom_sid global_sid_Creator_Owner_Domain =       /* Creator Owner domain */
 { 1, 0, {0,0,0,0,0,3}, {0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_NT_Authority =                /* NT Authority */
+const struct dom_sid global_sid_NT_Authority =                 /* NT Authority */
 { 1, 0, {0,0,0,0,0,5}, {0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_System =                      /* System */
+const struct dom_sid global_sid_System =                       /* System */
 { 1, 1, {0,0,0,0,0,5}, {18,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_NULL =                        /* NULL sid */
+const struct dom_sid global_sid_NULL =                         /* NULL sid */
 { 1, 1, {0,0,0,0,0,0}, {0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Authenticated_Users = /* All authenticated rids */
+const struct dom_sid global_sid_Authenticated_Users =  /* All authenticated rids */
 { 1, 1, {0,0,0,0,0,5}, {11,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
 #if 0
 /* for documentation */
-const DOM_SID global_sid_Restriced =                   /* Restriced Code */
+const struct dom_sid global_sid_Restriced =                    /* Restriced Code */
 { 1, 1, {0,0,0,0,0,5}, {12,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
 #endif
-const DOM_SID global_sid_Network =                     /* Network rids */
+const struct dom_sid global_sid_Network =                      /* Network rids */
 { 1, 1, {0,0,0,0,0,5}, {2,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
 
-const DOM_SID global_sid_Creator_Owner =               /* Creator Owner */
+const struct dom_sid global_sid_Creator_Owner =                /* Creator Owner */
 { 1, 1, {0,0,0,0,0,3}, {0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Creator_Group =               /* Creator Group */
+const struct dom_sid global_sid_Creator_Group =                /* Creator Group */
 { 1, 1, {0,0,0,0,0,3}, {1,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Anonymous =                   /* Anonymous login */
+const struct dom_sid global_sid_Anonymous =                    /* Anonymous login */
 { 1, 1, {0,0,0,0,0,5}, {7,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
 
-const DOM_SID global_sid_Builtin =                     /* Local well-known domain */
+const struct dom_sid global_sid_Builtin =                      /* Local well-known domain */
 { 1, 1, {0,0,0,0,0,5}, {32,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_Administrators =      /* Builtin administrators */
+const struct dom_sid global_sid_Builtin_Administrators =       /* Builtin administrators */
 { 1, 2, {0,0,0,0,0,5}, {32,544,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_Users =               /* Builtin users */
+const struct dom_sid global_sid_Builtin_Users =                /* Builtin users */
 { 1, 2, {0,0,0,0,0,5}, {32,545,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_Guests =              /* Builtin guest users */
+const struct dom_sid global_sid_Builtin_Guests =               /* Builtin guest users */
 { 1, 2, {0,0,0,0,0,5}, {32,546,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_Power_Users = /* Builtin power users */
+const struct dom_sid global_sid_Builtin_Power_Users =  /* Builtin power users */
 { 1, 2, {0,0,0,0,0,5}, {32,547,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_Account_Operators =   /* Builtin account operators */
+const struct dom_sid global_sid_Builtin_Account_Operators =    /* Builtin account operators */
 { 1, 2, {0,0,0,0,0,5}, {32,548,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_Server_Operators =    /* Builtin server operators */
+const struct dom_sid global_sid_Builtin_Server_Operators =     /* Builtin server operators */
 { 1, 2, {0,0,0,0,0,5}, {32,549,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_Print_Operators =     /* Builtin print operators */
+const struct dom_sid global_sid_Builtin_Print_Operators =      /* Builtin print operators */
 { 1, 2, {0,0,0,0,0,5}, {32,550,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_Backup_Operators =    /* Builtin backup operators */
+const struct dom_sid global_sid_Builtin_Backup_Operators =     /* Builtin backup operators */
 { 1, 2, {0,0,0,0,0,5}, {32,551,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_Replicator =          /* Builtin replicator */
+const struct dom_sid global_sid_Builtin_Replicator =           /* Builtin replicator */
 { 1, 2, {0,0,0,0,0,5}, {32,552,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Builtin_PreWin2kAccess =      /* Builtin pre win2k access */
+const struct dom_sid global_sid_Builtin_PreWin2kAccess =       /* Builtin pre win2k access */
 { 1, 2, {0,0,0,0,0,5}, {32,554,0,0,0,0,0,0,0,0,0,0,0,0,0}};
 
-const DOM_SID global_sid_Unix_Users =                  /* Unmapped Unix users */
+const struct dom_sid global_sid_Unix_Users =                   /* Unmapped Unix users */
 { 1, 1, {0,0,0,0,0,22}, {1,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
-const DOM_SID global_sid_Unix_Groups =                 /* Unmapped Unix groups */
+const struct dom_sid global_sid_Unix_Groups =                  /* Unmapped Unix groups */
 { 1, 1, {0,0,0,0,0,22}, {2,0,0,0,0,0,0,0,0,0,0,0,0,0,0}};
 
 /* Unused, left here for documentary purposes */
@@ -100,13 +103,13 @@ const DOM_SID global_sid_Unix_Groups =                    /* Unmapped Unix groups */
  * An NT compatible anonymous token.
  */
 
-static DOM_SID anon_sid_array[3] =
+static struct dom_sid anon_sid_array[3] =
 { { 1, 1, {0,0,0,0,0,1}, {0,0,0,0,0,0,0,0,0,0,0,0,0,0,0}},
   { 1, 1, {0,0,0,0,0,5}, {2,0,0,0,0,0,0,0,0,0,0,0,0,0,0}},
   { 1, 1, {0,0,0,0,0,5}, {7,0,0,0,0,0,0,0,0,0,0,0,0,0,0}} };
 NT_USER_TOKEN anonymous_token = { 3, anon_sid_array, SE_NONE };
 
-static DOM_SID system_sid_array[1] =
+static struct dom_sid system_sid_array[1] =
 { { 1, 1, {0,0,0,0,0,5}, {18,0,0,0,0,0,0,0,0,0,0,0,0,0,0}} };
 NT_USER_TOKEN system_token = { 1, system_sid_array, SE_ALL_PRIVS };
 
@@ -155,24 +158,11 @@ NT_USER_TOKEN *get_system_token(void)
        return &system_token;
 }
 
-/******************************************************************
- get the default domain/netbios name to be used when dealing 
- with our passdb list of accounts
-******************************************************************/
-
-const char *get_global_sam_name(void) 
-{
-       if ((lp_server_role() == ROLE_DOMAIN_PDC) || (lp_server_role() == ROLE_DOMAIN_BDC)) {
-               return lp_workgroup();
-       }
-       return global_myname();
-}
-
 /*****************************************************************
  Convert a SID to an ascii string.
 *****************************************************************/
 
-char *sid_to_fstring(fstring sidstr_out, const DOM_SID *sid)
+char *sid_to_fstring(fstring sidstr_out, const struct dom_sid *sid)
 {
        char *str = sid_string_talloc(talloc_tos(), sid);
        fstrcpy(sidstr_out, str);
@@ -181,14 +171,11 @@ char *sid_to_fstring(fstring sidstr_out, const DOM_SID *sid)
 }
 
 /*****************************************************************
- Essentially a renamed dom_sid_string from librpc/ndr with a
- panic if it didn't work
-
- This introduces a dependency on librpc/ndr/sid.o which can easily
- be turned around if necessary
+ Essentially a renamed dom_sid_string from
+ ../libcli/security/dom_sid.c with a panic if it didn't work.
 *****************************************************************/
 
-char *sid_string_talloc(TALLOC_CTX *mem_ctx, const DOM_SID *sid)
+char *sid_string_talloc(TALLOC_CTX *mem_ctx, const struct dom_sid *sid)
 {
        char *result = dom_sid_string(mem_ctx, sid);
        SMB_ASSERT(result != NULL);
@@ -199,16 +186,16 @@ char *sid_string_talloc(TALLOC_CTX *mem_ctx, const DOM_SID *sid)
  Useful function for debug lines.
 *****************************************************************/
 
-char *sid_string_dbg(const DOM_SID *sid)
+char *sid_string_dbg(const struct dom_sid *sid)
 {
-       return sid_string_talloc(debug_ctx(), sid);
+       return sid_string_talloc(talloc_tos(), sid);
 }
 
 /*****************************************************************
  Use with care!
 *****************************************************************/
 
-char *sid_string_tos(const DOM_SID *sid)
+char *sid_string_tos(const struct dom_sid *sid)
 {
        return sid_string_talloc(talloc_tos(), sid);
 }
@@ -216,36 +203,42 @@ char *sid_string_tos(const DOM_SID *sid)
 /*****************************************************************
  Convert a string to a SID. Returns True on success, False on fail.
 *****************************************************************/  
-   
-bool string_to_sid(DOM_SID *sidout, const char *sidstr)
+
+bool string_to_sid(struct dom_sid *sidout, const char *sidstr)
 {
        const char *p;
        char *q;
        /* BIG NOTE: this function only does SIDS where the identauth is not >= 2^32 */
        uint32 conv;
-  
+
        if ((sidstr[0] != 'S' && sidstr[0] != 's') || sidstr[1] != '-') {
-               DEBUG(3,("string_to_sid: Sid %s does not start with 'S-'.\n", sidstr));
-               return False;
+               goto format_error;
        }
 
        ZERO_STRUCTP(sidout);
 
        /* Get the revision number. */
        p = sidstr + 2;
+
+       if (!isdigit(*p)) {
+               goto format_error;
+       }
+
        conv = (uint32) strtoul(p, &q, 10);
        if (!q || (*q != '-')) {
-               DEBUG(3,("string_to_sid: Sid %s is not in a valid format.\n", sidstr));
-               return False;
+               goto format_error;
        }
        sidout->sid_rev_num = (uint8) conv;
        q++;
 
+       if (!isdigit(*q)) {
+               goto format_error;
+       }
+
        /* get identauth */
        conv = (uint32) strtoul(q, &q, 10);
        if (!q || (*q != '-')) {
-               DEBUG(0,("string_to_sid: Sid %s is not in a valid format.\n", sidstr));
-               return False;
+               goto format_error;
        }
        /* identauth in decimal should be <  2^32 */
        /* NOTE - the conv value is in big-endian format. */
@@ -259,36 +252,44 @@ bool string_to_sid(DOM_SID *sidout, const char *sidstr)
        q++;
        sidout->num_auths = 0;
 
-       for(conv = (uint32) strtoul(q, &q, 10);
-           q && (*q =='-' || *q =='\0') && (sidout->num_auths < MAXSUBAUTHS);
-           conv = (uint32) strtoul(q, &q, 10)) {
-               sid_append_rid(sidout, conv);
-               if (*q == '\0')
-                       break;
-               q++;
-       }
-               
-       return True;
-}
+       while (true) {
+               char *end;
 
-DOM_SID *string_sid_talloc(TALLOC_CTX *mem_ctx, const char *sidstr)
-{
-       DOM_SID *result = TALLOC_P(mem_ctx, DOM_SID);
+               if (!isdigit(*q)) {
+                       goto format_error;
+               }
 
-       if (result == NULL)
-               return NULL;
+               conv = strtoul(q, &end, 10);
+               if (end == q) {
+                       goto format_error;
+               }
 
-       if (!string_to_sid(result, sidstr))
-               return NULL;
+               if (!sid_append_rid(sidout, conv)) {
+                       DEBUG(3, ("Too many sid auths in %s\n", sidstr));
+                       return false;
+               }
 
-       return result;
+               q = end;
+               if (*q == '\0') {
+                       break;
+               }
+               if (*q != '-') {
+                       goto format_error;
+               }
+               q += 1;
+       }
+       return true;
+
+format_error:
+       DEBUG(3, ("string_to_sid: SID %s is not in a valid format\n", sidstr));
+       return false;
 }
 
 /*****************************************************************
  Add a rid to the end of a sid
 *****************************************************************/  
 
-bool sid_append_rid(DOM_SID *sid, uint32 rid)
+bool sid_append_rid(struct dom_sid *sid, uint32 rid)
 {
        if (sid->num_auths < MAXSUBAUTHS) {
                sid->sub_auths[sid->num_auths++] = rid;
@@ -297,7 +298,7 @@ bool sid_append_rid(DOM_SID *sid, uint32 rid)
        return False;
 }
 
-bool sid_compose(DOM_SID *dst, const DOM_SID *domain_sid, uint32 rid)
+bool sid_compose(struct dom_sid *dst, const struct dom_sid *domain_sid, uint32 rid)
 {
        sid_copy(dst, domain_sid);
        return sid_append_rid(dst, rid);
@@ -307,7 +308,7 @@ bool sid_compose(DOM_SID *dst, const DOM_SID *domain_sid, uint32 rid)
  Removes the last rid from the end of a sid
 *****************************************************************/  
 
-bool sid_split_rid(DOM_SID *sid, uint32 *rid)
+bool sid_split_rid(struct dom_sid *sid, uint32 *rid)
 {
        if (sid->num_auths > 0) {
                sid->num_auths--;
@@ -321,11 +322,11 @@ bool sid_split_rid(DOM_SID *sid, uint32 *rid)
  Return the last rid from the end of a sid
 *****************************************************************/  
 
-bool sid_peek_rid(const DOM_SID *sid, uint32 *rid)
+bool sid_peek_rid(const struct dom_sid *sid, uint32 *rid)
 {
        if (!sid || !rid)
                return False;           
-       
+
        if (sid->num_auths > 0) {
                *rid = sid->sub_auths[sid->num_auths - 1];
                return True;
@@ -338,11 +339,11 @@ bool sid_peek_rid(const DOM_SID *sid, uint32 *rid)
  and check the sid against the exp_dom_sid  
 *****************************************************************/  
 
-bool sid_peek_check_rid(const DOM_SID *exp_dom_sid, const DOM_SID *sid, uint32 *rid)
+bool sid_peek_check_rid(const struct dom_sid *exp_dom_sid, const struct dom_sid *sid, uint32 *rid)
 {
        if (!exp_dom_sid || !sid || !rid)
                return False;
-                       
+
        if (sid->num_auths != (exp_dom_sid->num_auths+1)) {
                return False;
        }
@@ -351,7 +352,7 @@ bool sid_peek_check_rid(const DOM_SID *exp_dom_sid, const DOM_SID *sid, uint32 *
                *rid=(-1);
                return False;
        }
-       
+
        return sid_peek_rid(sid, rid);
 }
 
@@ -359,7 +360,7 @@ bool sid_peek_check_rid(const DOM_SID *exp_dom_sid, const DOM_SID *sid, uint32 *
  Copies a sid
 *****************************************************************/  
 
-void sid_copy(DOM_SID *dst, const DOM_SID *src)
+void sid_copy(struct dom_sid *dst, const struct dom_sid *src)
 {
        int i;
 
@@ -378,7 +379,7 @@ void sid_copy(DOM_SID *dst, const DOM_SID *src)
  Write a sid out into on-the-wire format.
 *****************************************************************/  
 
-bool sid_linearize(char *outbuf, size_t len, const DOM_SID *sid)
+bool sid_linearize(char *outbuf, size_t len, const struct dom_sid *sid)
 {
        size_t i;
 
@@ -395,10 +396,10 @@ bool sid_linearize(char *outbuf, size_t len, const DOM_SID *sid)
 }
 
 /*****************************************************************
- Parse a on-the-wire SID to a DOM_SID.
+ Parse a on-the-wire SID to a struct dom_sid.
 *****************************************************************/  
 
-bool sid_parse(const char *inbuf, size_t len, DOM_SID *sid)
+bool sid_parse(const char *inbuf, size_t len, struct dom_sid *sid)
 {
        int i;
        if (len < 8)
@@ -420,7 +421,7 @@ bool sid_parse(const char *inbuf, size_t len, DOM_SID *sid)
  Compare the auth portion of two sids.
 *****************************************************************/  
 
-static int sid_compare_auth(const DOM_SID *sid1, const DOM_SID *sid2)
+static int sid_compare_auth(const struct dom_sid *sid1, const struct dom_sid *sid2)
 {
        int i;
 
@@ -445,7 +446,7 @@ static int sid_compare_auth(const DOM_SID *sid1, const DOM_SID *sid2)
  Compare two sids.
 *****************************************************************/  
 
-int sid_compare(const DOM_SID *sid1, const DOM_SID *sid2)
+int sid_compare(const struct dom_sid *sid1, const struct dom_sid *sid2)
 {
        int i;
 
@@ -472,7 +473,7 @@ int sid_compare(const DOM_SID *sid1, const DOM_SID *sid2)
  this just compares the leading sub-auths
 *****************************************************************/  
 
-int sid_compare_domain(const DOM_SID *sid1, const DOM_SID *sid2)
+int sid_compare_domain(const struct dom_sid *sid1, const struct dom_sid *sid2)
 {
        int n, i;
 
@@ -489,7 +490,7 @@ int sid_compare_domain(const DOM_SID *sid1, const DOM_SID *sid2)
  Compare two sids.
 *****************************************************************/  
 
-bool sid_equal(const DOM_SID *sid1, const DOM_SID *sid2)
+bool sid_equal(const struct dom_sid *sid1, const struct dom_sid *sid2)
 {
        return sid_compare(sid1, sid2) == 0;
 }
@@ -498,9 +499,9 @@ bool sid_equal(const DOM_SID *sid1, const DOM_SID *sid2)
  Returns true if SID is internal (and non-mappable).
 *****************************************************************/
 
-bool non_mappable_sid(DOM_SID *sid)
+bool non_mappable_sid(struct dom_sid *sid)
 {
-       DOM_SID dom;
+       struct dom_sid dom;
        uint32 rid;
 
        sid_copy(&dom, sid);
@@ -516,29 +517,31 @@ bool non_mappable_sid(DOM_SID *sid)
 }
 
 /*****************************************************************
- Return the binary string representation of a DOM_SID.
+ Return the binary string representation of a struct dom_sid.
  Caller must free.
 *****************************************************************/
 
-char *sid_binstring(const DOM_SID *sid)
+char *sid_binstring(TALLOC_CTX *mem_ctx, const struct dom_sid *sid)
 {
-       char *buf, *s;
+       uint8_t *buf;
+       char *s;
        int len = ndr_size_dom_sid(sid, 0);
-       buf = (char *)SMB_MALLOC(len);
-       if (!buf)
+       buf = talloc_array(mem_ctx, uint8_t, len);
+       if (!buf) {
                return NULL;
-       sid_linearize(buf, len, sid);
-       s = binary_string_rfc2254(buf, len);
-       free(buf);
+       }
+       sid_linearize((char *)buf, len, sid);
+       s = binary_string_rfc2254(mem_ctx, buf, len);
+       TALLOC_FREE(buf);
        return s;
 }
 
 /*****************************************************************
- Return the binary string representation of a DOM_SID.
+ Return the binary string representation of a struct dom_sid.
  Caller must free.
 *****************************************************************/
 
-char *sid_binstring_hex(const DOM_SID *sid)
+char *sid_binstring_hex(const struct dom_sid *sid)
 {
        char *buf, *s;
        int len = ndr_size_dom_sid(sid, 0);
@@ -555,17 +558,18 @@ char *sid_binstring_hex(const DOM_SID *sid)
  Tallocs a duplicate SID. 
 ********************************************************************/ 
 
-DOM_SID *sid_dup_talloc(TALLOC_CTX *ctx, const DOM_SID *src)
+struct dom_sid *sid_dup_talloc(TALLOC_CTX *ctx, const struct dom_sid *src)
 {
-       DOM_SID *dst;
-       
-       if(!src)
+       struct dom_sid *dst;
+
+       if (src == NULL) {
+               return NULL;
+       }
+       dst = talloc_zero(ctx, struct dom_sid);
+       if (dst == NULL) {
                return NULL;
-       
-       if((dst = TALLOC_ZERO_P(ctx, DOM_SID)) != NULL) {
-               sid_copy( dst, src);
        }
-       
+       sid_copy(dst, src);
        return dst;
 }
 
@@ -573,20 +577,20 @@ DOM_SID *sid_dup_talloc(TALLOC_CTX *ctx, const DOM_SID *src)
  Add SID to an array SIDs
 ********************************************************************/
 
-bool add_sid_to_array(TALLOC_CTX *mem_ctx, const DOM_SID *sid, 
-                     DOM_SID **sids, size_t *num)
+NTSTATUS add_sid_to_array(TALLOC_CTX *mem_ctx, const struct dom_sid *sid,
+                         struct dom_sid **sids, uint32_t *num)
 {
-       *sids = TALLOC_REALLOC_ARRAY(mem_ctx, *sids, DOM_SID,
+       *sids = TALLOC_REALLOC_ARRAY(mem_ctx, *sids, struct dom_sid,
                                             (*num)+1);
        if (*sids == NULL) {
                *num = 0;
-               return False;
+               return NT_STATUS_NO_MEMORY;
        }
 
        sid_copy(&((*sids)[*num]), sid);
        *num += 1;
 
-       return True;
+       return NT_STATUS_OK;
 }
 
 
@@ -594,14 +598,14 @@ bool add_sid_to_array(TALLOC_CTX *mem_ctx, const DOM_SID *sid,
  Add SID to an array SIDs ensuring that it is not already there
 ********************************************************************/
 
-bool add_sid_to_array_unique(TALLOC_CTX *mem_ctx, const DOM_SID *sid,
-                            DOM_SID **sids, size_t *num_sids)
+NTSTATUS add_sid_to_array_unique(TALLOC_CTX *mem_ctx, const struct dom_sid *sid,
+                                struct dom_sid **sids, uint32_t *num_sids)
 {
        size_t i;
 
        for (i=0; i<(*num_sids); i++) {
                if (sid_compare(sid, &(*sids)[i]) == 0)
-                       return True;
+                       return NT_STATUS_OK;
        }
 
        return add_sid_to_array(mem_ctx, sid, sids, num_sids);
@@ -611,9 +615,9 @@ bool add_sid_to_array_unique(TALLOC_CTX *mem_ctx, const DOM_SID *sid,
  Remove SID from an array
 ********************************************************************/
 
-void del_sid_from_array(const DOM_SID *sid, DOM_SID **sids, size_t *num)
+void del_sid_from_array(const struct dom_sid *sid, struct dom_sid **sids, size_t *num)
 {
-       DOM_SID *sid_list = *sids;
+       struct dom_sid *sid_list = *sids;
        size_t i;
 
        for ( i=0; i<*num; i++ ) {
@@ -632,7 +636,7 @@ void del_sid_from_array(const DOM_SID *sid, DOM_SID **sids, size_t *num)
 
        for ( ; i<*num; i++ ) 
                sid_copy( &sid_list[i], &sid_list[i+1] );
-       
+
        return;
 }
 
@@ -645,7 +649,7 @@ bool add_rid_to_array_unique(TALLOC_CTX *mem_ctx,
                if ((*pp_rids)[i] == rid)
                        return True;
        }
-       
+
        *pp_rids = TALLOC_REALLOC_ARRAY(mem_ctx, *pp_rids, uint32, *p_num+1);
 
        if (*pp_rids == NULL) {
@@ -658,54 +662,79 @@ bool add_rid_to_array_unique(TALLOC_CTX *mem_ctx,
        return True;
 }
 
-bool is_null_sid(const DOM_SID *sid)
+bool is_null_sid(const struct dom_sid *sid)
 {
-       static const DOM_SID null_sid = {0};
+       static const struct dom_sid null_sid = {0};
        return sid_equal(sid, &null_sid);
 }
 
+bool is_sid_in_token(const NT_USER_TOKEN *token, const struct dom_sid *sid)
+{
+        int i;
+
+        for (i=0; i<token->num_sids; i++) {
+                if (sid_compare(sid, &token->sids[i]) == 0)
+                        return true;
+        }
+        return false;
+}
+
 NTSTATUS sid_array_from_info3(TALLOC_CTX *mem_ctx,
-                             const NET_USER_INFO_3 *info3,
-                             DOM_SID **user_sids,
-                             size_t *num_user_sids,
-                             bool include_user_group_rid)
+                             const struct netr_SamInfo3 *info3,
+                             struct dom_sid **user_sids,
+                             uint32_t *num_user_sids,
+                             bool include_user_group_rid,
+                             bool skip_ressource_groups)
 {
-       DOM_SID sid;
-       DOM_SID *sid_array = NULL;
-       size_t num_sids = 0;
+       NTSTATUS status;
+       struct dom_sid sid;
+       struct dom_sid *sid_array = NULL;
+       uint32_t num_sids = 0;
        int i;
 
        if (include_user_group_rid) {
-
-               if (!sid_compose(&sid, &(info3->dom_sid.sid),
-                                info3->user_rid)
-                   || !add_sid_to_array(mem_ctx, &sid,
-                                        &sid_array, &num_sids)) {
-                       DEBUG(3,("could not add user SID from rid 0x%x\n",
-                                info3->user_rid));                     
+               if (!sid_compose(&sid, info3->base.domain_sid, info3->base.rid)) {
+                       DEBUG(3, ("could not compose user SID from rid 0x%x\n",
+                                 info3->base.rid));
                        return NT_STATUS_INVALID_PARAMETER;
                }
-
-               if (!sid_compose(&sid, &(info3->dom_sid.sid),
-                                info3->group_rid)
-                   || !add_sid_to_array(mem_ctx, &sid, 
-                                        &sid_array, &num_sids)) {
-                       DEBUG(3,("could not append additional group rid 0x%x\n",
-                                info3->group_rid));                    
-                       
-                       return NT_STATUS_INVALID_PARAMETER;
+               status = add_sid_to_array(mem_ctx, &sid, &sid_array, &num_sids);
+               if (!NT_STATUS_IS_OK(status)) {
+                       DEBUG(3, ("could not append user SID from rid 0x%x\n",
+                                 info3->base.rid));
+                       return status;
                }
        }
 
-       for (i = 0; i < info3->num_groups2; i++) {
-               if (!sid_compose(&sid, &(info3->dom_sid.sid),
-                                info3->gids[i].g_rid)
-                   || !add_sid_to_array(mem_ctx, &sid,
-                                        &sid_array, &num_sids)) {
-                       DEBUG(3,("could not append additional group rid 0x%x\n",
-                                info3->gids[i].g_rid));        
+       if (!sid_compose(&sid, info3->base.domain_sid, info3->base.primary_gid)) {
+               DEBUG(3, ("could not compose group SID from rid 0x%x\n",
+                         info3->base.primary_gid));
+               return NT_STATUS_INVALID_PARAMETER;
+       }
+       status = add_sid_to_array(mem_ctx, &sid, &sid_array, &num_sids);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(3, ("could not append group SID from rid 0x%x\n",
+                         info3->base.rid));
+               return status;
+       }
+
+       for (i = 0; i < info3->base.groups.count; i++) {
+               /* Don't add the primary group sid twice. */
+               if (info3->base.primary_gid == info3->base.groups.rids[i].rid) {
+                       continue;
+               }
+               if (!sid_compose(&sid, info3->base.domain_sid,
+                                info3->base.groups.rids[i].rid)) {
+                       DEBUG(3, ("could not compose SID from additional group "
+                                 "rid 0x%x\n", info3->base.groups.rids[i].rid));
                        return NT_STATUS_INVALID_PARAMETER;
                }
+               status = add_sid_to_array(mem_ctx, &sid, &sid_array, &num_sids);
+               if (!NT_STATUS_IS_OK(status)) {
+                       DEBUG(3, ("could not append SID from additional group "
+                                 "rid 0x%x\n", info3->base.groups.rids[i].rid));
+                       return status;
+               }
        }
 
        /* Copy 'other' sids.  We need to do sid filtering here to
@@ -714,12 +743,19 @@ NTSTATUS sid_array_from_info3(TALLOC_CTX *mem_ctx,
            http://www.microsoft.com/windows2000/techinfo/administration/security/sidfilter.asp
          */
 
-       for (i = 0; i < info3->num_other_sids; i++) {
-               if (!add_sid_to_array(mem_ctx, &info3->other_sids[i].sid,
-                                     &sid_array, &num_sids)) {
+       for (i = 0; i < info3->sidcount; i++) {
+
+               if (skip_ressource_groups &&
+                   (info3->sids[i].attributes & SE_GROUP_RESOURCE)) {
+                       continue;
+               }
+
+               status = add_sid_to_array(mem_ctx, info3->sids[i].sid,
+                                     &sid_array, &num_sids);
+               if (!NT_STATUS_IS_OK(status)) {
                        DEBUG(3, ("could not add SID to array: %s\n",
-                                 sid_string_dbg(&info3->other_sids[i].sid)));
-                       return NT_STATUS_NO_MEMORY;
+                                 sid_string_dbg(info3->sids[i].sid)));
+                       return status;
                }
        }