From cf2aa9251fbf19a0aafc7b9909a2f88b2a0c883f Mon Sep 17 00:00:00 2001 From: Garming Sam Date: Fri, 21 Mar 2014 09:10:52 +1300 Subject: [PATCH] param: remove unnecessary uses of lp_string_set Change-Id: I190e8b62dee24c998f388eae1cc9558e6b8bc925 Signed-off-by: Garming Sam Reviewed-by: Andrew Bartlett Reviewed-by: Michael Adam --- lib/param/loadparm.c | 33 +++++++++------------------------ 1 file changed, 9 insertions(+), 24 deletions(-) diff --git a/lib/param/loadparm.c b/lib/param/loadparm.c index 912363d0487..728377a480f 100644 --- a/lib/param/loadparm.c +++ b/lib/param/loadparm.c @@ -1082,15 +1082,9 @@ bool handle_realm(struct loadparm_context *lp_ctx, struct loadparm_service *serv return false; } - if (lp_ctx->s3_fns != NULL) { - lp_ctx->s3_fns->lp_string_set(ptr, pszParmValue); - lp_ctx->s3_fns->lp_string_set(&lp_ctx->globals->realm, upper); - lp_ctx->s3_fns->lp_string_set(&lp_ctx->globals->dnsdomain, lower); - } else { - lpcfg_string_set(lp_ctx, ptr, pszParmValue); - lpcfg_string_set(lp_ctx, &lp_ctx->globals->realm, upper); - lpcfg_string_set(lp_ctx, &lp_ctx->globals->dnsdomain, lower); - } + lpcfg_string_set(lp_ctx->globals->ctx, ptr, pszParmValue); + lpcfg_string_set(lp_ctx->globals->ctx, &lp_ctx->globals->realm, upper); + lpcfg_string_set(lp_ctx->globals->ctx, &lp_ctx->globals->dnsdomain, lower); return true; } @@ -1166,11 +1160,7 @@ bool handle_copy(struct loadparm_context *lp_ctx, struct loadparm_service *servi bool handle_debug_list(struct loadparm_context *lp_ctx, struct loadparm_service *service, const char *pszParmValue, char **ptr) { - if (lp_ctx->s3_fns != NULL) { - lp_ctx->s3_fns->lp_string_set(ptr, pszParmValue); - } else { - lpcfg_string_set(lp_ctx, ptr, pszParmValue); - } + lpcfg_string_set(lp_ctx->globals->ctx, ptr, pszParmValue); return debug_parse_levels(pszParmValue); } @@ -1178,13 +1168,12 @@ bool handle_debug_list(struct loadparm_context *lp_ctx, struct loadparm_service bool handle_logfile(struct loadparm_context *lp_ctx, struct loadparm_service *service, const char *pszParmValue, char **ptr) { - if (lp_ctx->s3_fns != NULL) { - lp_ctx->s3_fns->lp_string_set(ptr, pszParmValue); - } else { + if (lp_ctx->s3_fns == NULL) { debug_set_logfile(pszParmValue); - lpcfg_string_set(lp_ctx, ptr, pszParmValue); } + lpcfg_string_set(lp_ctx->globals->ctx, ptr, pszParmValue); + return true; } @@ -1197,16 +1186,14 @@ bool handle_charset(struct loadparm_context *lp_ctx, struct loadparm_service *se { if (lp_ctx->s3_fns) { if (*ptr == NULL || strcmp(*ptr, pszParmValue) != 0) { - lp_ctx->s3_fns->lp_string_set(ptr, pszParmValue); global_iconv_handle = smb_iconv_handle_reinit(NULL, lpcfg_dos_charset(lp_ctx), lpcfg_unix_charset(lp_ctx), true, global_iconv_handle); } - return true; } - return lpcfg_string_set(lp_ctx, ptr, pszParmValue); + return lpcfg_string_set(lp_ctx->globals->ctx, ptr, pszParmValue); } @@ -1243,16 +1230,14 @@ bool handle_dos_charset(struct loadparm_context *lp_ctx, struct loadparm_service DEFAULT_DOS_CHARSET)); pszParmValue = DEFAULT_DOS_CHARSET; } - lp_ctx->s3_fns->lp_string_set(ptr, pszParmValue); global_iconv_handle = smb_iconv_handle_reinit(NULL, lpcfg_dos_charset(lp_ctx), lpcfg_unix_charset(lp_ctx), true, global_iconv_handle); } - return true; } - return lpcfg_string_set(lp_ctx, ptr, pszParmValue); + return lpcfg_string_set(lp_ctx->globals->ctx, ptr, pszParmValue); } bool handle_printing(struct loadparm_context *lp_ctx, struct loadparm_service *service, -- 2.11.4.GIT