diff options
author | Garming Sam <garming@catalyst.net.nz> | 2014-02-27 16:31:14 +1300 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2014-07-07 23:32:36 +0200 |
commit | e1c50d9542a847361bcaf1bafca6691e56245873 (patch) | |
tree | 10c02bd71538835bd0f3ea6838b562c670f14b1b /source3/param | |
parent | 39fb65e54c4bf603c20ed01c1834b02e5be8c3c3 (diff) | |
download | samba-e1c50d9542a847361bcaf1bafca6691e56245873.tar.gz samba-e1c50d9542a847361bcaf1bafca6691e56245873.tar.xz samba-e1c50d9542a847361bcaf1bafca6691e56245873.zip |
param: simplify lp_do_parameter
No longer checks for globals vs service twice.
Change-Id: I065e732ddb1ca2dda63dd22e3caf291fce8d327b
Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Nadezhda Ivanova <nivanova@samba.org>
Diffstat (limited to 'source3/param')
-rw-r--r-- | source3/param/loadparm.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c index 70085d3404..03b2ea6d17 100644 --- a/source3/param/loadparm.c +++ b/source3/param/loadparm.c @@ -2617,6 +2617,7 @@ bool lp_do_parameter(int snum, const char *pszParmName, const char *pszParmValue /* we might point at a service, the default service or a global */ if (snum < 0) { parm_ptr = lp_parm_ptr(NULL, &parm_table[parmnum]); + mem_ctx = Globals.ctx; } else { if (parm_table[parmnum].p_class == P_GLOBAL) { DEBUG(0, @@ -2626,9 +2627,7 @@ bool lp_do_parameter(int snum, const char *pszParmName, const char *pszParmValue return true; } parm_ptr = lp_parm_ptr(ServicePtrs[snum], &parm_table[parmnum]); - } - if (snum >= 0) { if (!ServicePtrs[snum]->copymap) init_copymap(ServicePtrs[snum]); @@ -2640,9 +2639,8 @@ bool lp_do_parameter(int snum, const char *pszParmName, const char *pszParmValue bitmap_clear(ServicePtrs[snum]->copymap, i); } } + mem_ctx = ServicePtrs[snum]; - } else { - mem_ctx = Globals.ctx; } /* if it is a special case then go ahead */ |