diff options
| author | Michael Adam <obnox@samba.org> | 2008-04-09 09:54:17 +0200 |
|---|---|---|
| committer | Michael Adam <obnox@samba.org> | 2008-04-10 01:29:01 +0200 |
| commit | bb91ab6116055b3a450de3925737f91c3fdf4dca (patch) | |
| tree | 95b6789513c17f030a2944b17fcfe9b0248ccf67 /source3/lib/smbconf | |
| parent | d26daa02346952d2342fb92b89ce2ac96d0591ba (diff) | |
| download | samba-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)
Diffstat (limited to 'source3/lib/smbconf')
| -rw-r--r-- | source3/lib/smbconf/smbconf_reg.c | 10 |
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; |