summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Adam <obnox@samba.org>2008-04-03 15:29:25 +0200
committerMichael Adam <obnox@samba.org>2008-04-03 15:43:04 +0200
commit2830626877acdbfbaba342b8790cfbd1fe0dc0a5 (patch)
treeb9a88ed04de09f1c1f1219716a7a3807eebf60d4
parent24c54ee4fbe8d093d3f4d729ef2b055b82798f9a (diff)
downloadsamba-2830626877acdbfbaba342b8790cfbd1fe0dc0a5.tar.gz
samba-2830626877acdbfbaba342b8790cfbd1fe0dc0a5.tar.bz2
samba-2830626877acdbfbaba342b8790cfbd1fe0dc0a5.zip
registry: remove parameter checks from smbconf backend: they are in libsmbconf.
Michael (This used to be commit 118cf3813336122a060916848e37d2d5d25bff92)
-rw-r--r--source3/registry/reg_backend_smbconf.c192
1 files changed, 1 insertions, 191 deletions
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,