From 2830626877acdbfbaba342b8790cfbd1fe0dc0a5 Mon Sep 17 00:00:00 2001 From: Michael Adam Date: Thu, 3 Apr 2008 15:29:25 +0200 Subject: registry: remove parameter checks from smbconf backend: they are in libsmbconf. Michael (This used to be commit 118cf3813336122a060916848e37d2d5d25bff92) --- source3/registry/reg_backend_smbconf.c | 192 +-------------------------------- 1 file changed, 1 insertion(+), 191 deletions(-) (limited to 'source3') diff --git a/source3/registry/reg_backend_smbconf.c b/source3/registry/reg_backend_smbconf.c index a6e478200f..2e4a5f1c1d 100644 --- a/source3/registry/reg_backend_smbconf.c +++ b/source3/registry/reg_backend_smbconf.c @@ -40,199 +40,9 @@ static int smbconf_fetch_values( const char *key, REGVAL_CTR *val ) return regdb_ops.fetch_values(key, val); } -static WERROR regval_hilvl_to_lolvl(TALLOC_CTX *mem_ctx, const char *valname, - struct registry_value *src, - REGISTRY_VALUE **dst) -{ - WERROR err; - DATA_BLOB value_data; - REGISTRY_VALUE *newval = NULL; - - if (dst == NULL) { - return WERR_INVALID_PARAM; - } - - err = registry_push_value(mem_ctx, src, &value_data); - if (!W_ERROR_IS_OK(err)) { - DEBUG(10, ("error calling registry_push_value.\n")); - return err; - } - - newval = regval_compose(mem_ctx, valname, src->type, - (char *)value_data.data, value_data.length); - if (newval == NULL) { - DEBUG(10, ("error composing registry value. (no memory?)\n")); - return WERR_NOMEM; - } - - *dst = newval; - return WERR_OK; -} - -static WERROR regval_lolvl_to_hilvl(TALLOC_CTX *mem_ctx, REGISTRY_VALUE *src, - struct registry_value **dst) -{ - if (dst == NULL) { - return WERR_INVALID_PARAM; - } - - return registry_pull_value(mem_ctx, dst, regval_type(src), - regval_data_p(src), regval_size(src), - regval_size(src)); -} - -/* - * Utility function used by smbconf_store_values to canonicalize - * a registry value. - * registry_pull_value / registry_push_value are used for (un)marshalling. - */ -static REGISTRY_VALUE *smbconf_canonicalize_regval(TALLOC_CTX *mem_ctx, - REGISTRY_VALUE *theval) -{ - char *valstr; - size_t len; - const char *canon_valname; - const char *canon_valstr; - bool inverse; - struct registry_value *value; - WERROR err; - TALLOC_CTX *tmp_ctx; - REGISTRY_VALUE *newval = NULL; - - if (!lp_parameter_is_valid(regval_name(theval)) || - lp_parameter_is_canonical(regval_name(theval))) - { - return theval; - } - - tmp_ctx = talloc_stackframe(); - if (tmp_ctx == NULL) { - DEBUG(1, ("out of memory...\n")); - goto done; - } - - err = regval_lolvl_to_hilvl(tmp_ctx, theval, &value); - if (!W_ERROR_IS_OK(err)) { - goto done; - } - - /* we need the value-string zero-terminated */ - valstr = value->v.sz.str; - len = value->v.sz.len; - if (valstr[len - 1] != '\0') { - DEBUG(10, ("string is not '\\0'-terminated. adding '\\0'.\n")); - valstr = TALLOC_REALLOC_ARRAY(tmp_ctx, valstr, char, len + 1); - if (valstr == NULL) { - DEBUG(1, ("out of memory\n")); - goto done; - } - valstr[len] = '\0'; - } - - if (!lp_canonicalize_parameter(regval_name(theval), &canon_valname, - &inverse)) - { - DEBUG(5, ("Error: lp_canonicalize_parameter failed after " - "lp_parameter_is_valid. This should not happen!\n")); - goto done; - } - DEBUG(10, ("old value name: '%s', canonical value name: '%s'\n", - regval_name(theval), canon_valname)); - if (inverse && lp_string_is_valid_boolean(valstr)) { - lp_invert_boolean(valstr, &canon_valstr); - } else { - canon_valstr = valstr; - } - - ZERO_STRUCTP(value); - value->type = REG_SZ; - value->v.sz.str = CONST_DISCARD(char *, canon_valstr); - value->v.sz.len = strlen(canon_valstr) + 1; - - err = regval_hilvl_to_lolvl(mem_ctx, canon_valname, value, &newval); - if (!W_ERROR_IS_OK(err)) { - DEBUG(10, ("error calling regval_hilvl_to_lolvl.\n")); - goto done; - } - -done: - TALLOC_FREE(tmp_ctx); - return newval; -} - static bool smbconf_store_values( const char *key, REGVAL_CTR *val ) { - int i; - int num_values = regval_ctr_numvals(val); - REGVAL_CTR *new_val_ctr; - - /* - * we build a second regval container and copy over the values, - * possibly changing names to the canonical name, because when - * canonicalizing parameter names and replacing the original parameter - * (with reval_ctr_deletevalue and regval_ctr_addvalue) in the original - * container, the order would change and that is not so good in the - * "for" loop... :-o - */ - new_val_ctr = TALLOC_ZERO_P(val, REGVAL_CTR); - if (new_val_ctr == NULL) { - DEBUG(1, ("out of memory\n")); - return False; - } - - for (i=0; i < num_values; i++) { - REGISTRY_VALUE *theval = regval_ctr_specific_value(val, i); - const char *valname = regval_name(theval); - int res; - - DEBUG(10, ("inspecting value '%s'\n", valname)); - - /* unfortunately, we can not reject names that are not - * valid parameter names here, since e.g. regedit first - * creates values as "New Value #1" and so on and then - * drops into rename. */ - - if (regval_type(theval) != REG_SZ) { - DEBUG(1, ("smbconf_store_values: only registry value " - "type REG_SZ currently allowed under key " - "smbconf\n")); - return False; - } - - if (registry_smbconf_valname_forbidden(valname)) { - DEBUG(1, ("smbconf_store_values: value '%s' forbidden " - "in registry.\n", valname)); - return False; - } - - if (lp_parameter_is_valid(valname) && - !lp_parameter_is_canonical(valname)) - { - DEBUG(5, ("valid parameter '%s' given but it is a " - "synonym. going to canonicalize it.\n", - valname)); - theval = smbconf_canonicalize_regval(val, theval); - if (theval == NULL) { - DEBUG(10, ("error canonicalizing registry " - "value\n")); - return False; - } - } else { - DEBUG(10, ("%s parameter found, " - "copying it to new container...\n", - (lp_parameter_is_valid(valname)? - "valid":"unknown"))); - } - res = regval_ctr_copyvalue(new_val_ctr, theval); - if (res == 0) { - DEBUG(10, ("error calling regval_ctr_copyvalue. " - "(no memory?)\n")); - return False; - } - DEBUG(10, ("parameter copied. container now has %d values.\n", - res)); - } - return regdb_ops.store_values(key, new_val_ctr); + return regdb_ops.store_values(key, val); } static bool smbconf_reg_access_check(const char *keyname, uint32 requested, -- cgit