diff options
| author | Wilco Baan Hofman <wilco@baanhofman.nl> | 2010-05-19 18:31:19 +0200 |
|---|---|---|
| committer | Jelmer Vernooij <jelmer@samba.org> | 2010-06-20 17:19:12 +0200 |
| commit | 31aeddfa4354cbf0e0b562d0dcfe1f41cd95e0f9 (patch) | |
| tree | 52bfc5bbefbb49adeeec0e8e1da5c651aa487585 /source4 | |
| parent | 299531ce327340937e72530a96991c33fcd48e48 (diff) | |
| download | samba-31aeddfa4354cbf0e0b562d0dcfe1f41cd95e0f9.tar.gz samba-31aeddfa4354cbf0e0b562d0dcfe1f41cd95e0f9.tar.bz2 samba-31aeddfa4354cbf0e0b562d0dcfe1f41cd95e0f9.zip | |
Fix crash when get_gpo_info returns incorrect data.
Signed-off-by: Jelmer Vernooij <jelmer@samba.org>
Diffstat (limited to 'source4')
| -rw-r--r-- | source4/lib/policy/gp_ldap.c | 16 | ||||
| -rw-r--r-- | source4/lib/policy/gp_manage.c | 1 |
2 files changed, 12 insertions, 5 deletions
diff --git a/source4/lib/policy/gp_ldap.c b/source4/lib/policy/gp_ldap.c index b8b24d4c86..5ef161d12c 100644 --- a/source4/lib/policy/gp_ldap.c +++ b/source4/lib/policy/gp_ldap.c @@ -62,11 +62,18 @@ static NTSTATUS parse_gpo(TALLOC_CTX *mem_ctx, struct ldb_message *msg, struct g DEBUG(9, ("Parsing GPO LDAP data for %s\n", gpo->dn)); - gpo->display_name = talloc_steal(mem_ctx, ldb_msg_find_attr_as_string(msg, "displayName", "")); - gpo->name = talloc_steal(mem_ctx, ldb_msg_find_attr_as_string(msg, "name", "")); + gpo->display_name = ldb_msg_find_attr_as_string(msg, "displayName", ""); + gpo->name = ldb_msg_find_attr_as_string(msg, "name", ""); gpo->flags = ldb_msg_find_attr_as_uint(msg, "name", 0); gpo->version = ldb_msg_find_attr_as_uint(msg, "version", 0); - gpo->file_sys_path = talloc_steal(mem_ctx, ldb_msg_find_attr_as_string(msg, "gPCFileSysPath", "")); + gpo->file_sys_path = ldb_msg_find_attr_as_string(msg, "gPCFileSysPath", ""); + + if (gpo->display_name[0] != '\0') + gpo->display_name = talloc_steal(mem_ctx, gpo->display_name); + if (gpo->name[0] != '\0') + gpo->name = talloc_steal(mem_ctx, gpo->name); + if (gpo->file_sys_path[0] != '\0') + gpo->file_sys_path = talloc_steal(mem_ctx, gpo->file_sys_path); /* Pull the security descriptor through the NDR library */ data = ldb_msg_find_ldb_val(msg, "nTSecurityDescriptor"); @@ -792,8 +799,6 @@ NTSTATUS gp_create_ldap_gpo(struct gp_context *gp_ctx, struct gp_object *gpo) rv = ldb_msg_add_string(msg, "gpCFunctionalityVersion", "2"); if (rv != LDB_SUCCESS) goto ldb_msg_add_error; - /* FIXME Add security descriptor as well */ - rv = ldb_add(gp_ctx->ldb_ctx, msg); if (rv != LDB_SUCCESS) { DEBUG(0, ("LDB add error: %s\n", ldb_errstring(gp_ctx->ldb_ctx))); @@ -853,6 +858,7 @@ NTSTATUS gp_create_ldap_gpo(struct gp_context *gp_ctx, struct gp_object *gpo) return NT_STATUS_UNSUCCESSFUL; } + gpo->dn = talloc_steal(gpo, ldb_dn_get_linearized(gpo_dn)); talloc_free(mem_ctx); return NT_STATUS_OK; diff --git a/source4/lib/policy/gp_manage.c b/source4/lib/policy/gp_manage.c index 7313ab2557..580f87116d 100644 --- a/source4/lib/policy/gp_manage.c +++ b/source4/lib/policy/gp_manage.c @@ -134,6 +134,7 @@ NTSTATUS gp_create_gpo (struct gp_context *gp_ctx, const char *display_name, str name[i] = toupper(name[i]); } + gpo->dn = NULL; gpo->name = name; gpo->flags = 0; gpo->version = 0; |