diff options
| author | Stefan Metzmacher <metze@samba.org> | 2018-02-01 23:08:08 +0100 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2018-09-05 13:35:24 +0200 |
| commit | 467e6aeca631369a436bd5d3d96690f3552c3d67 (patch) | |
| tree | f770f42cbde6eeedfe5eeb1f4f5e01162a87ac2f | |
| parent | 8e81aa4ec37e0f45a338d5741cca93bf7094ae84 (diff) | |
| download | samba-467e6aeca631369a436bd5d3d96690f3552c3d67.tar.gz | |
dsdb/util_trusts: domain_dn is an input parameter of dsdb_trust_crossref_tdo_info()
We should not overwrite it within the function.
Currently it doesn't matter as we don't have multiple domains
within our forest, but that will change in future.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=11517
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
(cherry picked from commit f5f96f558b499770cdeb3d38998167a387e058b9)
| -rw-r--r-- | source4/dsdb/common/util_trusts.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/source4/dsdb/common/util_trusts.c b/source4/dsdb/common/util_trusts.c index a083d869f6e..bb34aa1054b 100644 --- a/source4/dsdb/common/util_trusts.c +++ b/source4/dsdb/common/util_trusts.c @@ -481,12 +481,6 @@ static NTSTATUS dsdb_trust_crossref_tdo_info(TALLOC_CTX *mem_ctx, *_trust_parent_tdo = NULL; } - domain_dn = ldb_get_default_basedn(sam_ctx); - if (domain_dn == NULL) { - TALLOC_FREE(frame); - return NT_STATUS_INTERNAL_ERROR; - } - partitions_dn = samdb_partitions_dn(sam_ctx, frame); if (partitions_dn == NULL) { TALLOC_FREE(frame); |