diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2014-05-23 16:06:17 +1200 |
|---|---|---|
| committer | Karolin Seeger <kseeger@samba.org> | 2014-07-15 12:46:15 +0200 |
| commit | f72899ec2bc0ad3f6cd1e3d586235543b51f7cd1 (patch) | |
| tree | 1a68b5d71b7db5f49ec832bf73ba46919e1d3c12 /source4/libnet/libnet_vampire.c | |
| parent | 4730d74e38998aaeadbe13682cc6462c3ceb4766 (diff) | |
| download | samba-f72899ec2bc0ad3f6cd1e3d586235543b51f7cd1.tar.gz | |
dsdb: Do not store a struct ldb_dn in struct schema_data
The issue is that the DN contains a pointer to the ldb it belongs to,
and if this is not kept around long enough, we might reference memory
after it is de-allocated.
Andrew Bartlett
Change-Id: I040a6c37a3164b3309f370e32e598dd56b1a1bbb
Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
(cherry picked from commit 8327321225251e312ccbd06bbefa5ebf98099f34)
Diffstat (limited to 'source4/libnet/libnet_vampire.c')
| -rw-r--r-- | source4/libnet/libnet_vampire.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/source4/libnet/libnet_vampire.c b/source4/libnet/libnet_vampire.c index 9489f0bccb..aff9d08e49 100644 --- a/source4/libnet/libnet_vampire.c +++ b/source4/libnet/libnet_vampire.c @@ -508,11 +508,6 @@ NTSTATUS libnet_vampire_cb_schema_chunk(void *private_data, s->self_made_schema = dsdb_new_schema(s); NT_STATUS_HAVE_NO_MEMORY(s->self_made_schema); - s->self_made_schema->base_dn = ldb_dn_new(s->self_made_schema, - s->ldb, - c->forest->schema_dn_str); - NT_STATUS_HAVE_NO_MEMORY(s->self_made_schema->base_dn); - status = dsdb_load_prefixmap_from_drsuapi(s->self_made_schema, mapping_ctr); if (!W_ERROR_IS_OK(status)) { return werror_to_ntstatus(status); |