>From 2d8110d0a011092661249fc5fd19b7a007c21464 Mon Sep 17 00:00:00 2001 From: Noel Power Date: Tue, 11 Apr 2017 11:26:45 +0100 Subject: [PATCH] Check for valid values for global conf 'name resolve order' variable This variable is populated by a list of values where each value should be a known option. This patch ensures that illegal values are detected. Signed-off-by: Noel Power --- docs-xml/smbdotconf/protocol/nameresolveorder.xml | 1 + lib/param/loadparm.c | 38 ++++++++++++++++++++++- lib/param/loadparm.h | 1 + source3/param/loadparm.c | 5 ++- 4 files changed, 43 insertions(+), 2 deletions(-) diff --git a/docs-xml/smbdotconf/protocol/nameresolveorder.xml b/docs-xml/smbdotconf/protocol/nameresolveorder.xml index ec3aaf3..1e04582 100644 --- a/docs-xml/smbdotconf/protocol/nameresolveorder.xml +++ b/docs-xml/smbdotconf/protocol/nameresolveorder.xml @@ -1,6 +1,7 @@ This option is used by the programs in the Samba diff --git a/lib/param/loadparm.c b/lib/param/loadparm.c index 6aa757f..7786e4b 100644 --- a/lib/param/loadparm.c +++ b/lib/param/loadparm.c @@ -69,6 +69,7 @@ #include "tdb.h" #include "librpc/gen_ndr/nbt.h" #include "libds/common/roles.h" +#include "lib/util/samba_util.h" #ifdef HAVE_HTTPCONNECTENCRYPT #include @@ -1669,6 +1670,39 @@ static bool set_variable_helper(TALLOC_CTX *mem_ctx, int parmnum, void *parm_ptr } +bool handle_name_resolve_order(struct loadparm_context *lp_ctx, + struct loadparm_service *service, + const char *pszParmValue, char **ptr) +{ + const char **valid_values = + str_list_make_v3_const(NULL, + DEFAULT_NAME_RESOLVE_ORDER, + NULL); + const char **values_to_set = + str_list_make_v3_const(lp_ctx->globals->ctx, + pszParmValue, + NULL); + int i; + bool value_is_valid = false; + TALLOC_FREE(lp_ctx->globals->name_resolve_order); + for (i = 0; values_to_set[i] != NULL; i++) { + value_is_valid = str_list_check(valid_values, values_to_set[i]); + if (!value_is_valid) { + DBG_ERR("WARNING: Ignoring invalid list value '%s' " + "for parameter 'name resolve order'\n", + values_to_set[i]); + break; + } + } + if (value_is_valid) { + lp_ctx->globals->name_resolve_order = values_to_set; + } else { + TALLOC_FREE(values_to_set); + } + TALLOC_FREE(valid_values); + return value_is_valid; +} + static bool set_variable(TALLOC_CTX *mem_ctx, struct loadparm_service *service, int parmnum, void *parm_ptr, const char *pszParmName, const char *pszParmValue, @@ -2562,7 +2596,9 @@ struct loadparm_context *loadparm_init(TALLOC_CTX *mem_ctx) myname = get_myname(lp_ctx); lpcfg_do_global_parameter(lp_ctx, "netbios name", myname); talloc_free(myname); - lpcfg_do_global_parameter(lp_ctx, "name resolve order", "lmhosts wins host bcast"); + lpcfg_do_global_parameter(lp_ctx, + "name resolve order", + DEFAULT_NAME_RESOLVE_ORDER); lpcfg_do_global_parameter(lp_ctx, "fstype", "NTFS"); diff --git a/lib/param/loadparm.h b/lib/param/loadparm.h index f9fb7d8..37c0ebf 100644 --- a/lib/param/loadparm.h +++ b/lib/param/loadparm.h @@ -105,6 +105,7 @@ struct file_lists { time_t modtime; }; +#define DEFAULT_NAME_RESOLVE_ORDER "lmhosts wins host bcast" #define FLAG_DEPRECATED 0x1000 /* options that should no longer be used */ #define FLAG_SYNONYM 0x2000 /* options that is a synonym of another option */ #define FLAG_CMDLINE 0x10000 /* option has been overridden */ diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c index d4ef1c1..7726ea5 100644 --- a/source3/param/loadparm.c +++ b/source3/param/loadparm.c @@ -609,7 +609,10 @@ static void init_globals(struct loadparm_context *lp_ctx, bool reinit_globals) lpcfg_string_set(Globals.ctx, &Globals.logon_path, "\\\\%N\\%U\\profile"); - Globals.name_resolve_order = str_list_make_v3_const(NULL, "lmhosts wins host bcast", NULL); + Globals.name_resolve_order = + str_list_make_v3_const(NULL, + DEFAULT_NAME_RESOLVE_ORDER, + NULL); lpcfg_string_set(Globals.ctx, &Globals.password_server, "*"); Globals.algorithmic_rid_base = BASE_RID; -- 2.12.0