summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Adam <obnox@samba.org>2008-04-09 09:54:17 +0200
committerMichael Adam <obnox@samba.org>2008-04-10 01:29:01 +0200
commitbb91ab6116055b3a450de3925737f91c3fdf4dca (patch)
tree95b6789513c17f030a2944b17fcfe9b0248ccf67
parentd26daa02346952d2342fb92b89ce2ac96d0591ba (diff)
downloadsamba-bb91ab6116055b3a450de3925737f91c3fdf4dca.tar.gz
samba-bb91ab6116055b3a450de3925737f91c3fdf4dca.tar.bz2
samba-bb91ab6116055b3a450de3925737f91c3fdf4dca.zip
libsmbconf: prevent getting/deleting value "includes".
This has to be handled differently (by using get_includes / set_includes) Michael (This used to be commit 5a880c6a2f2415220557a76a9b4ce9a17c766819)
-rw-r--r--source3/lib/smbconf/smbconf_reg.c10
1 files changed, 10 insertions, 0 deletions
diff --git a/source3/lib/smbconf/smbconf_reg.c b/source3/lib/smbconf/smbconf_reg.c
index 75b76a8de0..fbc8577835 100644
--- a/source3/lib/smbconf/smbconf_reg.c
+++ b/source3/lib/smbconf/smbconf_reg.c
@@ -884,6 +884,11 @@ static WERROR smbconf_reg_get_parameter(struct smbconf_ctx *ctx,
goto done;
}
+ if (!smbconf_reg_valname_valid(param)) {
+ werr = WERR_INVALID_PARAM;
+ goto done;
+ }
+
if (!smbconf_value_exists(key, param)) {
werr = WERR_INVALID_PARAM;
goto done;
@@ -923,6 +928,11 @@ static WERROR smbconf_reg_delete_parameter(struct smbconf_ctx *ctx,
goto done;
}
+ if (!smbconf_reg_valname_valid(param)) {
+ werr = WERR_INVALID_PARAM;
+ goto done;
+ }
+
if (!smbconf_value_exists(key, param)) {
werr = WERR_INVALID_PARAM;
goto done;