diff options
| author | Stefan Metzmacher <metze@samba.org> | 2010-07-10 09:06:44 +0200 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2010-07-10 09:35:02 +0200 |
| commit | 1ea4215e5681ccef921bbf77bb1af79fc1a82b77 (patch) | |
| tree | b8a850c569c96517d4dc01efc3c04a02dac922c7 /source4/libnet/libnet_group.c | |
| parent | 35c554bef02e9acfe9cbec12d1f4ebc1067bedd4 (diff) | |
| download | samba-1ea4215e5681ccef921bbf77bb1af79fc1a82b77.tar.gz samba-1ea4215e5681ccef921bbf77bb1af79fc1a82b77.tar.bz2 samba-1ea4215e5681ccef921bbf77bb1af79fc1a82b77.zip | |
s4:libnet_GroupList: allocate children strings on the correct talloc parent
Otherwise the _recv() function won't move the children strings to the
callers memory context and let the callers crash.
metze
Diffstat (limited to 'source4/libnet/libnet_group.c')
| -rw-r--r-- | source4/libnet/libnet_group.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source4/libnet/libnet_group.c b/source4/libnet/libnet_group.c index 8c88aa38bb..7679b42610 100644 --- a/source4/libnet/libnet_group.c +++ b/source4/libnet/libnet_group.c @@ -671,11 +671,11 @@ static void continue_groups_enumerated(struct tevent_req *subreq) if (composite_nomem(group_sid, c)) return; /* groupname */ - s->groups[i].groupname = talloc_strdup(c, entry->name.string); + s->groups[i].groupname = talloc_strdup(s->groups, entry->name.string); if (composite_nomem(s->groups[i].groupname, c)) return; /* sid string */ - s->groups[i].sid = dom_sid_string(c, group_sid); + s->groups[i].sid = dom_sid_string(s->groups, group_sid); if (composite_nomem(s->groups[i].sid, c)) return; } |