[crash fix] don't use already free'ed memory
authorStefan Metzmacher <metze@samba.org>
Wed, 24 Oct 2007 12:29:06 +0000 (14:29 +0200)
committerStefan Metzmacher <metze@samba.org>
Wed, 24 Oct 2007 13:36:53 +0000 (15:36 +0200)
(found by "make valgrindtest" and my "start winbindd on make test" patch)

metze
(cherry picked from commit fe21e48489852720a05b305b251e4f5cbb200f7a)
(cherry picked from commit 26d8a1ad20c10da495970c584983fbd261b4946e)
(This used to be commit a128a8805e172738334ec6854548f138c335058b)

source3/groupdb/mapping.c

index bd0d775fc50d3da66562f80fd19e4b75b3b385d0..6f54e3d550628e9885b5a4caba37b621490269f9 100644 (file)
@@ -705,12 +705,12 @@ NTSTATUS pdb_create_builtin_alias(uint32 rid)
                return NT_STATUS_ACCESS_DENIED;
        }
 
-       DEBUG(10,("Creating alias %s with gid %d\n", name, gid));
+       DEBUG(10,("Creating alias %s with gid %d\n", groupname, gid));
 
        map.gid = gid;
        sid_copy(&map.sid, &sid);
        map.sid_name_use = SID_NAME_ALIAS;
-       fstrcpy(map.nt_name, name);
+       fstrcpy(map.nt_name, groupname);
        fstrcpy(map.comment, "");
 
        status = pdb_add_group_mapping_entry(&map);