Remove more redundant lsa parsing functions.
[bbaumbach/samba-autobuild/.git] / source / rpc_parse / parse_sec.c
index c2c8ad120c9848b136e7a74a959baf718cefcda7..6ea128d3a49cf61c9ecc902ea7d54d889cd4a5fd 100644 (file)
@@ -3,13 +3,13 @@
  *  Version 1.9.
  *  RPC Pipe client / server routines
  *  Copyright (C) Andrew Tridgell              1992-1998,
- *  Copyright (C) Jeremy R. Allison            1995-1998
+ *  Copyright (C) Jeremy R. Allison            1995-2005.
  *  Copyright (C) Luke Kenneth Casson Leighton 1996-1998,
  *  Copyright (C) Paul Ashton                  1997-1998.
  *  
  *  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 2 of the License, or
+ *  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,
  *  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, write to the Free Software
- *  Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ *  along with this program; if not, see <http://www.gnu.org/licenses/>.
  */
 
-
 #include "includes.h"
 
-extern int DEBUGLEVEL;
-
-#define SD_HEADER_SIZE 0x14
-
-/*******************************************************************
- Sets up a SEC_ACCESS structure.
-********************************************************************/
-
-void init_sec_access(SEC_ACCESS *t, uint32 mask)
-{
-       t->mask = mask;
-}
+#undef DBGC_CLASS
+#define DBGC_CLASS DBGC_RPC_PARSE
 
 /*******************************************************************
  Reads or writes a SEC_ACCESS structure.
 ********************************************************************/
 
-BOOL sec_io_access(char *desc, SEC_ACCESS *t, prs_struct *ps, int depth)
+bool sec_io_access(const char *desc, SEC_ACCESS *t, prs_struct *ps, int depth)
 {
        if (t == NULL)
                return False;
 
        prs_debug(ps, depth, desc, "sec_io_access");
        depth++;
-
-       if(!prs_align(ps))
-               return False;
        
-       if(!prs_uint32("mask", ps, depth, &(t->mask)))
+       if(!prs_uint32("mask", ps, depth, t))
                return False;
 
        return True;
 }
 
-
-/*******************************************************************
- Sets up a SEC_ACE structure.
-********************************************************************/
-
-void init_sec_ace(SEC_ACE *t, DOM_SID *sid, uint8 type, SEC_ACCESS mask, uint8 flag)
-{
-       t->type = type;
-       t->flags = flag;
-       t->size = sid_size(sid) + 8;
-       t->info = mask;
-
-       ZERO_STRUCTP(&t->sid);
-       sid_copy(&t->sid, sid);
-}
-
 /*******************************************************************
  Reads or writes a SEC_ACE structure.
 ********************************************************************/
 
-BOOL sec_io_ace(char *desc, SEC_ACE *psa, prs_struct *ps, int depth)
+bool sec_io_ace(const char *desc, SEC_ACE *psa, prs_struct *ps, int depth)
 {
        uint32 old_offset;
        uint32 offset_ace_size;
+       uint8 type;
 
        if (psa == NULL)
                return False;
 
        prs_debug(ps, depth, desc, "sec_io_ace");
        depth++;
-
-       if(!prs_align(ps))
-               return False;
        
        old_offset = prs_offset(ps);
 
-       if(!prs_uint8("type ", ps, depth, &psa->type))
-               return False;
+       if (MARSHALLING(ps)) {
+               type = (uint8)psa->type;
+       }
 
-       if(!prs_uint8("flags", ps, depth, &psa->flags))
+       if(!prs_uint8("type ", ps, depth, &type))
                return False;
 
-       if(!prs_uint16_pre("size ", ps, depth, &psa->size, &offset_ace_size))
-               return False;
-
-       if(!sec_io_access("info ", &psa->info, ps, depth))
-               return False;
+       if (UNMARSHALLING(ps)) {
+               psa->type = (enum security_ace_type)type;
+       }
 
-       if(!prs_align(ps))
+       if(!prs_uint8("flags", ps, depth, &psa->flags))
                return False;
 
-       if(!smb_io_dom_sid("sid  ", &psa->sid , ps, depth))
+       if(!prs_uint16_pre("size ", ps, depth, &psa->size, &offset_ace_size))
                return False;
 
-       if(!prs_uint16_post("size ", ps, depth, &psa->size, offset_ace_size, old_offset))
+       if(!prs_uint32("access_mask", ps, depth, &psa->access_mask))
                return False;
 
-       return True;
-}
-
-/*******************************************************************
- Create a SEC_ACL structure.  
-********************************************************************/
-
-SEC_ACL *make_sec_acl(uint16 revision, int num_aces, SEC_ACE *ace_list)
-{
-       SEC_ACL *dst;
-       int i;
-
-       if((dst = (SEC_ACL *)malloc(sizeof(SEC_ACL))) == NULL)
-               return NULL;
-
-       ZERO_STRUCTP(dst);
+       /* check whether object access is present */
+       if (!sec_ace_object(psa->type)) {
+               if (!smb_io_dom_sid("trustee  ", &psa->trustee , ps, depth))
+                       return False;
+       } else {
+               if (!prs_uint32("obj_flags", ps, depth, &psa->object.object.flags))
+                       return False;
 
-       dst->revision = revision;
-       dst->num_aces = num_aces;
-       dst->size = 8;
+               if (psa->object.object.flags & SEC_ACE_OBJECT_PRESENT)
+                       if (!smb_io_uuid("obj_guid", &psa->object.object.type.type, ps,depth))
+                               return False;
 
-       if((dst->ace = (SEC_ACE *)malloc( sizeof(SEC_ACE) * num_aces )) == NULL) {
-               free_sec_acl(&dst);
-               return NULL;
-       }
+               if (psa->object.object.flags & SEC_ACE_OBJECT_INHERITED_PRESENT)
+                       if (!smb_io_uuid("inh_guid", &psa->object.object.inherited_type.inherited_type, ps,depth))
+                               return False;
 
-       for (i = 0; i < num_aces; i++) {
-               dst->ace[i] = ace_list[i]; /* Structure copy. */
-               dst->size += ace_list[i].size;
+               if(!smb_io_dom_sid("trustee  ", &psa->trustee , ps, depth))
+                       return False;
        }
 
-       return dst;
-}
+       /* Theorectically an ACE can have a size greater than the
+          sum of its components. When marshalling, pad with extra null bytes up to the
+          correct size. */
 
-/*******************************************************************
- Duplicate a SEC_ACL structure.  
-********************************************************************/
-
-SEC_ACL *dup_sec_acl( SEC_ACL *src)
-{
-       if(src == NULL)
-               return NULL;
-
-       return make_sec_acl( src->revision, src->num_aces, src->ace);
-}
-
-/*******************************************************************
- Delete a SEC_ACL structure.  
-********************************************************************/
-
-void free_sec_acl(SEC_ACL **ppsa)
-{
-       SEC_ACL *psa;
+       if (MARSHALLING(ps) && (psa->size > prs_offset(ps) - old_offset)) {
+               uint32 extra_len = psa->size - (prs_offset(ps) - old_offset);
+               uint32 i;
+               uint8 c = 0;
 
-       if(ppsa == NULL || *ppsa == NULL)
-               return;
+               for (i = 0; i < extra_len; i++) {
+                       if (!prs_uint8("ace extra space", ps, depth, &c))
+                               return False;
+               }
+       }
 
-       psa = *ppsa;
-       if (psa->ace != NULL)
-               free(psa->ace);
+       if(!prs_uint16_post("size ", ps, depth, &psa->size, offset_ace_size, old_offset))
+               return False;
 
-       free(psa);
-       *ppsa = NULL;
+       return True;
 }
 
 /*******************************************************************
@@ -188,12 +130,20 @@ void free_sec_acl(SEC_ACL **ppsa)
  for you as it reads them.
 ********************************************************************/
 
-BOOL sec_io_acl(char *desc, SEC_ACL **ppsa, prs_struct *ps, int depth)
+bool sec_io_acl(const char *desc, SEC_ACL **ppsa, prs_struct *ps, int depth)
 {
-       int i;
+       unsigned int i;
        uint32 old_offset;
        uint32 offset_acl_size;
        SEC_ACL *psa;
+       uint16 revision;
+
+       /*
+        * Note that the size is always a multiple of 4 bytes due to the
+        * nature of the data structure.  Therefore the prs_align() calls
+        * have been removed as they through us off when doing two-layer
+        * marshalling such as in the printing code (RPC_BUFFER).  --jerry
+        */
 
        if (ppsa == NULL)
                return False;
@@ -204,424 +154,82 @@ BOOL sec_io_acl(char *desc, SEC_ACL **ppsa, prs_struct *ps, int depth)
                /*
                 * This is a read and we must allocate the stuct to read into.
                 */
-               if((psa = (SEC_ACL *)malloc(sizeof(SEC_ACL))) == NULL)
+               if((psa = PRS_ALLOC_MEM(ps, SEC_ACL, 1)) == NULL)
                        return False;
-               ZERO_STRUCTP(psa);
                *ppsa = psa;
        }
 
        prs_debug(ps, depth, desc, "sec_io_acl");
        depth++;
-
-       if(!prs_align(ps))
-               return False;
        
        old_offset = prs_offset(ps);
 
-       if(!prs_uint16("revision", ps, depth, &psa->revision))
+       if (MARSHALLING(ps)) {
+               revision = (uint16)psa->revision;
+       }
+
+       if(!prs_uint16("revision", ps, depth, &revision))
                return False;
 
+       if (UNMARSHALLING(ps)) {
+               psa->revision = (enum security_acl_revision)revision;
+       }
+
        if(!prs_uint16_pre("size     ", ps, depth, &psa->size, &offset_acl_size))
                return False;
 
        if(!prs_uint32("num_aces ", ps, depth, &psa->num_aces))
                return False;
 
-       if (UNMARSHALLING(ps) && psa->num_aces != 0) {
-               /* reading */
-               if((psa->ace = malloc(sizeof(psa->ace[0]) * psa->num_aces)) == NULL)
-                       return False;
-               ZERO_STRUCTP(psa->ace);
+       if (UNMARSHALLING(ps)) {
+               if (psa->num_aces) {
+                       if((psa->aces = PRS_ALLOC_MEM(ps, SEC_ACE, psa->num_aces)) == NULL)
+                               return False;
+               } else {
+                       psa->aces = NULL;
+               }
        }
 
        for (i = 0; i < psa->num_aces; i++) {
                fstring tmp;
                slprintf(tmp, sizeof(tmp)-1, "ace_list[%02d]: ", i);
-               if(!sec_io_ace(tmp, &psa->ace[i], ps, depth))
+               if(!sec_io_ace(tmp, &psa->aces[i], ps, depth))
                        return False;
        }
 
-       if(!prs_align(ps))
-               return False;
-
-       if(!prs_uint16_post("size     ", ps, depth, &psa->size, offset_acl_size, old_offset))
-               return False;
-
-       return True;
-}
-
-/*******************************************************************
- Works out the linearization size of a SEC_DESC.
-********************************************************************/
-
-size_t sec_desc_size(SEC_DESC *psd)
-{
-       size_t offset;
-
-       if (!psd) return 0;
-
-       offset = SD_HEADER_SIZE;
-
-       if (psd->owner_sid != NULL)
-               offset += ((sid_size(psd->owner_sid) + 3) & ~3);
-
-       if (psd->grp_sid != NULL)
-               offset += ((sid_size(psd->grp_sid) + 3) & ~3);
-
-       if (psd->sacl != NULL)
-               offset += ((psd->sacl->size + 3) & ~3);
-
-       if (psd->dacl != NULL)
-               offset += ((psd->dacl->size + 3) & ~3);
-
-       return offset;
-}
-
-/*******************************************************************
- Compares two SEC_ACE structures
-********************************************************************/
-
-BOOL sec_ace_equal(SEC_ACE *s1, SEC_ACE *s2)
-{
-       /* Trivial case */
-
-       if (!s1 && !s2) return True;
-
-       /* Check top level stuff */
-
-       if (s1->type != s2->type || s1->flags != s2->flags ||
-           s1->info.mask != s2->info.mask) {
-               return False;
-       }
-
-       /* Check SID */
-
-       if (!sid_equal(&s1->sid, &s2->sid)) {
-               return False;
-       }
-
-       return True;
-}
-
-/*******************************************************************
- Compares two SEC_ACL structures
-********************************************************************/
-
-BOOL sec_acl_equal(SEC_ACL *s1, SEC_ACL *s2)
-{
-       int i, j;
-
-       /* Trivial case */
-
-       if (!s1 && !s2) return True;
-
-       /* Check top level stuff */
-
-       if (s1->revision != s2->revision) {
-               DEBUG(10, ("sec_acl_equal(): revision differs (%d != %d)\n",
-                          s1->revision, s2->revision));
-               return False;
-       }
-
-       if (s1->num_aces != s2->num_aces) {
-               DEBUG(10, ("sec_acl_equal(): num_aces differs (%d != %d)\n",
-                          s1->revision, s2->revision));
-               return False;
-       }
-
-       /* The ACEs could be in any order so check each ACE in s1 against 
-          each ACE in s2. */
+       /* Theorectically an ACL can have a size greater than the
+          sum of its components. When marshalling, pad with extra null bytes up to the
+          correct size. */
 
-       for (i = 0; i < s1->num_aces; i++) {
-               BOOL found = False;
+       if (MARSHALLING(ps) && (psa->size > prs_offset(ps) - old_offset)) {
+               uint32 extra_len = psa->size - (prs_offset(ps) - old_offset);
+               uint8 c = 0;
 
-               for (j = 0; j < s2->num_aces; j++) {
-                       if (sec_ace_equal(&s1->ace[i], &s2->ace[j])) {
-                               found = True;
-                               break;
-                       }
+               for (i = 0; i < extra_len; i++) {
+                       if (!prs_uint8("acl extra space", ps, depth, &c))
+                               return False;
                }
-
-               if (!found) return False;
        }
 
-       return True;
-}
-
-/*******************************************************************
- Compares two SEC_DESC structures
-********************************************************************/
-
-BOOL sec_desc_equal(SEC_DESC *s1, SEC_DESC *s2)
-{
-       /* Trivial case */
-
-       if (!s1 && !s2) {
-               goto done;
-       }
-
-       /* Check top level stuff */
-
-       if (s1->revision != s2->revision) {
-               DEBUG(10, ("sec_desc_equal(): revision differs (%d != %d)\n",
-                          s1->revision, s2->revision));
-               return False;
-       }
-
-       if (s1->type!= s2->type) {
-               DEBUG(10, ("sec_desc_equal(): type differs (%d != %d)\n",
-                          s1->type, s2->type));
-               return False;
-       }
-
-       /* Check owner and group */
-
-       if (!sid_equal(s1->owner_sid, s2->owner_sid)) {
-               fstring str1, str2;
-
-               sid_to_string(str1, s1->owner_sid);
-               sid_to_string(str2, s2->owner_sid);
-
-               DEBUG(10, ("sec_desc_equal(): owner differs (%s != %s)\n",
-                          str1, str2));
-               return False;
-       }
-
-       if (!sid_equal(s1->grp_sid, s2->grp_sid)) {
-               fstring str1, str2;
-
-               sid_to_string(str1, s1->grp_sid);
-               sid_to_string(str2, s2->grp_sid);
-
-               DEBUG(10, ("sec_desc_equal(): group differs (%s != %s)\n",
-                          str1, str2));
-               return False;
-       }
-
-       /* Check ACLs present in one but not the other */
-
-       if ((s1->dacl && !s2->dacl) || (!s1->dacl && s2->dacl) ||
-           (s1->sacl && !s2->sacl) || (!s1->sacl && s2->sacl)) {
-               DEBUG(10, ("sec_desc_equal(): dacl or sacl not present\n"));
-               return False;
-       }
-
-       /* Sigh - we have to do it the hard way by iterating over all
-          the ACEs in the ACLs */
-
-       if (!sec_acl_equal(s1->dacl, s2->dacl) ||
-           !sec_acl_equal(s1->sacl, s2->sacl)) {
-               DEBUG(10, ("sec_desc_equal(): dacl/sacl list not equal\n"));
+       if(!prs_uint16_post("size     ", ps, depth, &psa->size, offset_acl_size, old_offset))
                return False;
-       }
 
- done:
-       DEBUG(10, ("sec_desc_equal(): secdescs are identical\n"));
        return True;
 }
 
-/*******************************************************************
- Merge part of security descriptor old_sec in to the empty sections of 
- security descriptor new_sec.
-********************************************************************/
-
-SEC_DESC_BUF *sec_desc_merge(SEC_DESC_BUF *new_sdb, SEC_DESC_BUF *old_sdb)
-{
-       DOM_SID *owner_sid, *group_sid;
-       SEC_DESC_BUF *return_sdb;
-       SEC_ACL *dacl, *sacl;
-       SEC_DESC *psd = NULL;
-       uint16 secdesc_type;
-       size_t secdesc_size;
-
-       /* Copy over owner and group sids.  There seems to be no flag for
-          this so just check the pointer values. */
-
-       owner_sid = new_sdb->sec->owner_sid ? new_sdb->sec->owner_sid :
-               old_sdb->sec->owner_sid;
-
-       group_sid = new_sdb->sec->grp_sid ? new_sdb->sec->grp_sid :
-               old_sdb->sec->grp_sid;
-       
-       secdesc_type = new_sdb->sec->type;
-
-       /* Ignore changes to the system ACL.  This has the effect of making
-          changes through the security tab audit button not sticking. 
-          Perhaps in future Samba could implement these settings somehow. */
-
-       sacl = NULL;
-       secdesc_type &= ~SEC_DESC_SACL_PRESENT;
-
-       /* Copy across discretionary ACL */
-
-       if (secdesc_type & SEC_DESC_DACL_PRESENT) {
-               dacl = new_sdb->sec->dacl;
-       } else {
-               dacl = old_sdb->sec->dacl;
-       }
-
-       /* Create new security descriptor from bits */
-
-       psd = make_sec_desc(new_sdb->sec->revision, 
-                           owner_sid, group_sid, sacl, dacl, &secdesc_size);
-
-       return_sdb = make_sec_desc_buf(secdesc_size, psd);
-
-       free_sec_desc(&psd);
-
-       return(return_sdb);
-}
-
-/*******************************************************************
- Creates a SEC_DESC structure
-********************************************************************/
-
-SEC_DESC *make_sec_desc(uint16 revision, 
-                       DOM_SID *owner_sid, DOM_SID *grp_sid,
-                       SEC_ACL *sacl, SEC_ACL *dacl, size_t *sd_size)
-{
-       SEC_DESC *dst;
-       uint32 offset;
-
-       *sd_size = 0;
-
-       if(( dst = (SEC_DESC *)malloc(sizeof(SEC_DESC))) == NULL)
-               return NULL;
-
-       ZERO_STRUCTP(dst);
-
-       dst->revision = revision;
-       dst->type     = SEC_DESC_SELF_RELATIVE;
-
-       if (sacl) dst->type |= SEC_DESC_SACL_PRESENT;
-       if (dacl) dst->type |= SEC_DESC_DACL_PRESENT;
-
-       dst->off_owner_sid = 0;
-       dst->off_grp_sid   = 0;
-       dst->off_sacl      = 0;
-       dst->off_dacl      = 0;
-
-       if(owner_sid && ((dst->owner_sid = sid_dup(owner_sid)) == NULL))
-               goto error_exit;
-
-       if(grp_sid && ((dst->grp_sid = sid_dup(grp_sid)) == NULL))
-               goto error_exit;
-
-       if(sacl && ((dst->sacl = dup_sec_acl(sacl)) == NULL))
-               goto error_exit;
-
-       if(dacl && ((dst->dacl = dup_sec_acl(dacl)) == NULL))
-               goto error_exit;
-               
-       offset = 0;
-
-       /*
-        * Work out the linearization sizes.
-        */
-
-       if (dst->owner_sid != NULL) {
-
-               if (offset == 0)
-                       offset = SD_HEADER_SIZE;
-
-               dst->off_owner_sid = offset;
-               offset += ((sid_size(dst->owner_sid) + 3) & ~3);
-       }
-
-       if (dst->grp_sid != NULL) {
-
-               if (offset == 0)
-                       offset = SD_HEADER_SIZE;
-
-               dst->off_grp_sid = offset;
-               offset += ((sid_size(dst->grp_sid) + 3) & ~3);
-       }
-
-       if (dst->sacl != NULL) {
-
-               if (offset == 0)
-                       offset = SD_HEADER_SIZE;
-
-               dst->off_sacl = offset;
-               offset += ((sacl->size + 3) & ~3);
-       }
-
-       if (dst->dacl != NULL) {
-
-               if (offset == 0)
-                       offset = SD_HEADER_SIZE;
-
-               dst->off_dacl = offset;
-               offset += ((dacl->size + 3) & ~3);
-       }
-
-       *sd_size = (size_t)((offset == 0) ? SD_HEADER_SIZE : offset);
-       return dst;
-
-error_exit:
-
-       *sd_size = 0;
-       free_sec_desc(&dst);
-       return NULL;
-}
-
-/*******************************************************************
- Duplicate a SEC_DESC structure.  
-********************************************************************/
-
-SEC_DESC *dup_sec_desc( SEC_DESC *src)
-{
-       size_t dummy;
-
-       if(src == NULL)
-               return NULL;
-
-       return make_sec_desc( src->revision, 
-                               src->owner_sid, src->grp_sid, src->sacl,
-                               src->dacl, &dummy);
-}
-
-/*******************************************************************
- Deletes a SEC_DESC structure
-********************************************************************/
-
-void free_sec_desc(SEC_DESC **ppsd)
-{
-       SEC_DESC *psd;
-
-       if(ppsd == NULL || *ppsd == NULL)
-               return;
-
-       psd = *ppsd;
-
-       free_sec_acl(&psd->dacl);
-       free_sec_acl(&psd->dacl);
-       free(psd->owner_sid);
-       free(psd->grp_sid);
-       free(psd);
-       *ppsd = NULL;
-}
-
-/*******************************************************************
- Creates a SEC_DESC structure with typical defaults.
-********************************************************************/
-
-SEC_DESC *make_standard_sec_desc(DOM_SID *owner_sid, DOM_SID *grp_sid,
-                                SEC_ACL *dacl, size_t *sd_size)
-{
-       return make_sec_desc(SEC_DESC_REVISION,
-                            owner_sid, grp_sid, NULL, dacl, sd_size);
-}
-
 /*******************************************************************
  Reads or writes a SEC_DESC structure.
  If reading and the *ppsd = NULL, allocates the structure.
 ********************************************************************/
 
-BOOL sec_io_desc(char *desc, SEC_DESC **ppsd, prs_struct *ps, int depth)
+bool sec_io_desc(const char *desc, SEC_DESC **ppsd, prs_struct *ps, int depth)
 {
        uint32 old_offset;
        uint32 max_offset = 0; /* after we're done, move offset to end */
+       uint32 tmp_offset = 0;
+       uint32 off_sacl, off_dacl, off_owner_sid, off_grp_sid;
+       uint16 revision;
+
        SEC_DESC *psd;
 
        if (ppsd == NULL)
@@ -631,9 +239,8 @@ BOOL sec_io_desc(char *desc, SEC_DESC **ppsd, prs_struct *ps, int depth)
 
        if (psd == NULL) {
                if(UNMARSHALLING(ps)) {
-                       if((psd = (SEC_DESC *)malloc(sizeof(SEC_DESC))) == NULL)
+                       if((psd = PRS_ALLOC_MEM(ps,SEC_DESC,1)) == NULL)
                                return False;
-                       ZERO_STRUCTP(psd);
                        *ppsd = psd;
                } else {
                        /* Marshalling - just ignore. */
@@ -644,163 +251,156 @@ BOOL sec_io_desc(char *desc, SEC_DESC **ppsd, prs_struct *ps, int depth)
        prs_debug(ps, depth, desc, "sec_io_desc");
        depth++;
 
-       if(!prs_align(ps))
-               return False;
-       
        /* start of security descriptor stored for back-calc offset purposes */
        old_offset = prs_offset(ps);
 
-       if(!prs_uint16("revision ", ps, depth, &psd->revision))
+       if (MARSHALLING(ps)) {
+               revision = (uint16)psd->revision;
+       }
+
+       if(!prs_uint16("revision", ps, depth, &revision))
                return False;
 
+       if (UNMARSHALLING(ps)) {
+               psd->revision = (enum security_descriptor_revision)revision;
+       }
+
        if(!prs_uint16("type     ", ps, depth, &psd->type))
                return False;
 
-       if(!prs_uint32("off_owner_sid", ps, depth, &psd->off_owner_sid))
+       if (MARSHALLING(ps)) {
+               uint32 offset = SEC_DESC_HEADER_SIZE;
+
+               /*
+                * Work out the offsets here, as we write it out.
+                */
+
+               if (psd->sacl != NULL) {
+                       off_sacl = offset;
+                       offset += psd->sacl->size;
+               } else {
+                       off_sacl = 0;
+               }
+
+               if (psd->dacl != NULL) {
+                       off_dacl = offset;
+                       offset += psd->dacl->size;
+               } else {
+                       off_dacl = 0;
+               }
+
+               if (psd->owner_sid != NULL) {
+                       off_owner_sid = offset;
+                       offset += ndr_size_dom_sid(psd->owner_sid, 0);
+               } else {
+                       off_owner_sid = 0;
+               }
+
+               if (psd->group_sid != NULL) {
+                       off_grp_sid = offset;
+                       offset += ndr_size_dom_sid(psd->group_sid, 0);
+               } else {
+                       off_grp_sid = 0;
+               }
+       }
+
+       if(!prs_uint32("off_owner_sid", ps, depth, &off_owner_sid))
                return False;
 
-       if(!prs_uint32("off_grp_sid  ", ps, depth, &psd->off_grp_sid))
+       if(!prs_uint32("off_grp_sid  ", ps, depth, &off_grp_sid))
                return False;
 
-       if(!prs_uint32("off_sacl     ", ps, depth, &psd->off_sacl))
+       if(!prs_uint32("off_sacl     ", ps, depth, &off_sacl))
                return False;
 
-       if(!prs_uint32("off_dacl     ", ps, depth, &psd->off_dacl))
+       if(!prs_uint32("off_dacl     ", ps, depth, &off_dacl))
                return False;
 
        max_offset = MAX(max_offset, prs_offset(ps));
 
-       if (psd->off_owner_sid != 0) {
+       if (off_owner_sid != 0) {
+
+               tmp_offset = prs_offset(ps);
+               if(!prs_set_offset(ps, old_offset + off_owner_sid))
+                       return False;
 
                if (UNMARSHALLING(ps)) {
-                       if(!prs_set_offset(ps, old_offset + psd->off_owner_sid))
-                               return False;
                        /* reading */
-                       if((psd->owner_sid = malloc(sizeof(*psd->owner_sid))) == NULL)
+                       if((psd->owner_sid = PRS_ALLOC_MEM(ps,DOM_SID,1)) == NULL)
                                return False;
-                       ZERO_STRUCTP(psd->owner_sid);
                }
 
                if(!smb_io_dom_sid("owner_sid ", psd->owner_sid , ps, depth))
                        return False;
-               if(!prs_align(ps))
+
+               max_offset = MAX(max_offset, prs_offset(ps));
+
+               if (!prs_set_offset(ps,tmp_offset))
                        return False;
        }
 
-       max_offset = MAX(max_offset, prs_offset(ps));
+       if (psd->group_sid != 0) {
 
-       if (psd->off_grp_sid != 0) {
+               tmp_offset = prs_offset(ps);
+               if(!prs_set_offset(ps, old_offset + off_grp_sid))
+                       return False;
 
                if (UNMARSHALLING(ps)) {
                        /* reading */
-                       if(!prs_set_offset(ps, old_offset + psd->off_grp_sid))
-                               return False;
-                       if((psd->grp_sid = malloc(sizeof(*psd->grp_sid))) == NULL)
+                       if((psd->group_sid = PRS_ALLOC_MEM(ps,DOM_SID,1)) == NULL)
                                return False;
-                       ZERO_STRUCTP(psd->grp_sid);
                }
 
-               if(!smb_io_dom_sid("grp_sid", psd->grp_sid, ps, depth))
+               if(!smb_io_dom_sid("grp_sid", psd->group_sid, ps, depth))
                        return False;
-               if(!prs_align(ps))
+                       
+               max_offset = MAX(max_offset, prs_offset(ps));
+
+               if (!prs_set_offset(ps,tmp_offset))
                        return False;
        }
 
-       max_offset = MAX(max_offset, prs_offset(ps));
-
-       if ((psd->type & SEC_DESC_SACL_PRESENT) && psd->off_sacl) {
-               if(!prs_set_offset(ps, old_offset + psd->off_sacl))
+       if ((psd->type & SEC_DESC_SACL_PRESENT) && off_sacl) {
+               tmp_offset = prs_offset(ps);
+               if(!prs_set_offset(ps, old_offset + off_sacl))
                        return False;
                if(!sec_io_acl("sacl", &psd->sacl, ps, depth))
                        return False;
-               if(!prs_align(ps))
+               max_offset = MAX(max_offset, prs_offset(ps));
+               if (!prs_set_offset(ps,tmp_offset))
                        return False;
        }
 
-       max_offset = MAX(max_offset, prs_offset(ps));
-
-       if ((psd->type & SEC_DESC_DACL_PRESENT) && psd->off_dacl != 0) {
-               if(!prs_set_offset(ps, old_offset + psd->off_dacl))
+       if ((psd->type & SEC_DESC_DACL_PRESENT) && off_dacl != 0) {
+               tmp_offset = prs_offset(ps);
+               if(!prs_set_offset(ps, old_offset + off_dacl))
                        return False;
                if(!sec_io_acl("dacl", &psd->dacl, ps, depth))
                        return False;
-               if(!prs_align(ps))
+               max_offset = MAX(max_offset, prs_offset(ps));
+               if (!prs_set_offset(ps,tmp_offset))
                        return False;
        }
 
-       max_offset = MAX(max_offset, prs_offset(ps));
-
        if(!prs_set_offset(ps, max_offset))
                return False;
-       return True;
-}
-
-/*******************************************************************
- Creates a SEC_DESC_BUF structure.
-********************************************************************/
-
-SEC_DESC_BUF *make_sec_desc_buf(size_t len, SEC_DESC *sec_desc)
-{
-       SEC_DESC_BUF *dst;
-
-       if((dst = (SEC_DESC_BUF *)malloc(sizeof(SEC_DESC_BUF))) == NULL)
-               return NULL;
-
-       ZERO_STRUCTP(dst);
 
-       /* max buffer size (allocated size) */
-       dst->max_len = (uint32)len;
-       dst->len = (uint32)len;
-
-       if(sec_desc && ((dst->sec = dup_sec_desc(sec_desc)) == NULL)) {
-               free_sec_desc_buf(&dst);
-               return NULL;
-       }
-
-       return dst;
-}
-
-/*******************************************************************
- Duplicates a SEC_DESC_BUF structure.
-********************************************************************/
-
-SEC_DESC_BUF *dup_sec_desc_buf(SEC_DESC_BUF *src)
-{
-       if(src == NULL)
-               return NULL;
-
-       return make_sec_desc_buf( src->len, src->sec);
-}
-
-/*******************************************************************
- Deletes a SEC_DESC_BUF structure.
-********************************************************************/
-
-void free_sec_desc_buf(SEC_DESC_BUF **ppsdb)
-{
-       SEC_DESC_BUF *psdb;
-
-       if(ppsdb == NULL || *ppsdb == NULL)
-               return;
-
-       psdb = *ppsdb;
-       free_sec_desc(&psdb->sec);
-       free(psdb);
-       *ppsdb = NULL;
+       return True;
 }
 
-
 /*******************************************************************
  Reads or writes a SEC_DESC_BUF structure.
 ********************************************************************/
 
-BOOL sec_io_desc_buf(char *desc, SEC_DESC_BUF **ppsdb, prs_struct *ps, int depth)
+bool sec_io_desc_buf(const char *desc, SEC_DESC_BUF **ppsdb, prs_struct *ps, int depth)
 {
        uint32 off_len;
        uint32 off_max_len;
        uint32 old_offset;
        uint32 size;
+       uint32 len;
        SEC_DESC_BUF *psdb;
+       uint32 ptr;
 
        if (ppsdb == NULL)
                return False;
@@ -808,9 +408,8 @@ BOOL sec_io_desc_buf(char *desc, SEC_DESC_BUF **ppsdb, prs_struct *ps, int depth
        psdb = *ppsdb;
 
        if (UNMARSHALLING(ps) && psdb == NULL) {
-               if((psdb = (SEC_DESC_BUF *)malloc(sizeof(SEC_DESC_BUF))) == NULL)
+               if((psdb = PRS_ALLOC_MEM(ps,SEC_DESC_BUF,1)) == NULL)
                        return False;
-               ZERO_STRUCTP(psdb);
                *ppsdb = psdb;
        }
 
@@ -820,20 +419,22 @@ BOOL sec_io_desc_buf(char *desc, SEC_DESC_BUF **ppsdb, prs_struct *ps, int depth
        if(!prs_align(ps))
                return False;
        
-       if(!prs_uint32_pre("max_len", ps, depth, &psdb->max_len, &off_max_len))
+       if(!prs_uint32_pre("max_len", ps, depth, &psdb->sd_size, &off_max_len))
                return False;
 
-       if(!prs_uint32    ("undoc  ", ps, depth, &psdb->undoc))
+       ptr = 1;
+       if(!prs_uint32    ("ptr  ", ps, depth, &ptr))
                return False;
 
-       if(!prs_uint32_pre("len    ", ps, depth, &psdb->len, &off_len))
+       len = ndr_size_security_descriptor(psdb->sd, 0);
+       if(!prs_uint32_pre("len    ", ps, depth, &len, &off_len))
                return False;
 
        old_offset = prs_offset(ps);
 
        /* reading, length is non-zero; writing, descriptor is non-NULL */
-       if ((UNMARSHALLING(ps) && psdb->len != 0) || (MARSHALLING(ps) && psdb->sec != NULL)) {
-               if(!sec_io_desc("sec   ", &psdb->sec, ps, depth))
+       if ((UNMARSHALLING(ps) && psdb->sd_size != 0) || (MARSHALLING(ps) && psdb->sd != NULL)) {
+               if(!sec_io_desc("sec   ", &psdb->sd, ps, depth))
                        return False;
        }
 
@@ -841,10 +442,10 @@ BOOL sec_io_desc_buf(char *desc, SEC_DESC_BUF **ppsdb, prs_struct *ps, int depth
                return False;
        
        size = prs_offset(ps) - old_offset;
-       if(!prs_uint32_post("max_len", ps, depth, &psdb->max_len, off_max_len, size == 0 ? psdb->max_len : size))
+       if(!prs_uint32_post("max_len", ps, depth, &psdb->sd_size, off_max_len, size == 0 ? psdb->sd_size : size))
                return False;
 
-       if(!prs_uint32_post("len    ", ps, depth, &psdb->len, off_len, size))
+       if(!prs_uint32_post("len    ", ps, depth, &len, off_len, size))
                return False;
 
        return True;