From 1627fcda3eff6e636438c90d07ffcfd497ea68e2 Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Wed, 12 Sep 2012 18:07:58 +1000 Subject: [PATCH] dns_server: Remove parameter 'dns recursive queries' and base this on 'dns forwarder' This simplifies a very common configuration. Andrew Bartlett Signed-off-by: Stefan Metzmacher --- lib/param/loadparm.c | 1 - lib/param/param_functions.c | 1 - lib/param/param_table.c | 8 -------- source4/dns_server/dns_server.c | 5 +++-- 4 files changed, 3 insertions(+), 12 deletions(-) diff --git a/lib/param/loadparm.c b/lib/param/loadparm.c index 277a92124b3..990dd294b91 100644 --- a/lib/param/loadparm.c +++ b/lib/param/loadparm.c @@ -2215,7 +2215,6 @@ struct loadparm_context *loadparm_init(TALLOC_CTX *mem_ctx) lpcfg_do_global_parameter(lp_ctx, "nsupdate command", "/usr/bin/nsupdate -g"); lpcfg_do_global_parameter(lp_ctx, "allow dns updates", "secure only"); - lpcfg_do_global_parameter(lp_ctx, "dns recursive queries", "False"); lpcfg_do_global_parameter(lp_ctx, "dns forwarder", ""); for (i = 0; parm_table[i].label; i++) { diff --git a/lib/param/param_functions.c b/lib/param/param_functions.c index bf6863e76d5..ce2f671d738 100644 --- a/lib/param/param_functions.c +++ b/lib/param/param_functions.c @@ -179,7 +179,6 @@ FN_GLOBAL_BOOL(defer_sharing_violations, bDeferSharingViolations) FN_GLOBAL_BOOL(disable_netbios, bDisableNetbios) FN_GLOBAL_BOOL(_disable_spoolss, bDisableSpoolss) FN_GLOBAL_BOOL(_domain_logons, bDomainLogons) -FN_GLOBAL_BOOL(dns_recursive_queries, dns_recursive_queries) FN_GLOBAL_BOOL(enable_asu_support, bASUSupport) FN_GLOBAL_BOOL(enable_core_files, bEnableCoreFiles) FN_GLOBAL_BOOL(enable_privileges, bEnablePrivileges) diff --git a/lib/param/param_table.c b/lib/param/param_table.c index cb9596c5f7d..325f2953423 100644 --- a/lib/param/param_table.c +++ b/lib/param/param_table.c @@ -4261,14 +4261,6 @@ static struct parm_struct parm_table[] = { .flags = FLAG_ADVANCED, }, { - .label = "dns recursive queries", - .type = P_BOOL, - .p_class = P_GLOBAL, - .offset = GLOBAL_VAR(dns_recursive_queries), - .special = NULL, - .enum_list = NULL - }, - { .label = "dns update command", .type = P_CMDLIST, .p_class = P_GLOBAL, diff --git a/source4/dns_server/dns_server.c b/source4/dns_server/dns_server.c index be1fecc008c..c88ea83f3f9 100644 --- a/source4/dns_server/dns_server.c +++ b/source4/dns_server/dns_server.c @@ -117,7 +117,7 @@ static struct tevent_req *dns_process_send(TALLOC_CTX *mem_ctx, struct dns_process_state *state; enum ndr_err_code ndr_err; WERROR ret; - + const char *forwarder = lpcfg_dns_forwarder(dns->task->lp_ctx); req = tevent_req_create(mem_ctx, &state, struct dns_process_state); if (req == NULL) { return NULL; @@ -156,7 +156,8 @@ static struct tevent_req *dns_process_send(TALLOC_CTX *mem_ctx, state->state.flags = state->in_packet.operation; state->state.flags |= DNS_FLAG_REPLY; - if (lpcfg_dns_recursive_queries(dns->task->lp_ctx)) { + + if (forwarder && *forwarder) { state->state.flags |= DNS_FLAG_RECURSION_AVAIL; } -- 2.11.4.GIT