diff options
author | Jelmer Vernooij <jelmer@samba.org> | 2007-09-27 19:49:53 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 15:07:22 -0500 |
commit | 5e2f9cd8e223368d38d49cf60f199bbd818b8732 (patch) | |
tree | 43263022c04d69104bf400dae79f0e00f18c33b7 /source4 | |
parent | 3bf358c4d1016c4c927cc31ab4dcd44130a07967 (diff) | |
download | samba-5e2f9cd8e223368d38d49cf60f199bbd818b8732.tar.gz samba-5e2f9cd8e223368d38d49cf60f199bbd818b8732.tar.bz2 samba-5e2f9cd8e223368d38d49cf60f199bbd818b8732.zip |
r25379: Use loadparm context parameter in a lot more places.
(This used to be commit 091961b13be665061c7e88ab4e2808c015bc403e)
Diffstat (limited to 'source4')
-rw-r--r-- | source4/param/loadparm.c | 36 | ||||
-rw-r--r-- | source4/param/loadparm.h | 1 | ||||
-rw-r--r-- | source4/param/param.h | 2 | ||||
-rw-r--r-- | source4/param/share_classic.c | 6 | ||||
-rw-r--r-- | source4/scripting/ejs/smbcalls_config.c | 6 | ||||
-rw-r--r-- | source4/utils/testparm.c | 18 |
6 files changed, 38 insertions, 31 deletions
diff --git a/source4/param/loadparm.c b/source4/param/loadparm.c index f914d706cd..1582eb6075 100644 --- a/source4/param/loadparm.c +++ b/source4/param/loadparm.c @@ -66,6 +66,8 @@ static bool bLoaded = false; +struct loadparm_context *global_loadparm = NULL; + #define standard_sub_basic talloc_strdup static bool do_parameter(const char *, const char *, void *); @@ -2293,6 +2295,8 @@ bool loadparm_init(struct loadparm_context *lp_ctx) int i; char *myname; + lp_ctx->bInGlobalSection = true; + DEBUG(3, ("Initialising global parameters\n")); for (i = 0; parm_table[i].label; i++) { @@ -2458,6 +2462,8 @@ bool lp_load(void) struct param_opt *data; struct loadparm_context *lp_ctx = &loadparm; + global_loadparm = lp_ctx; + bRetval = false; if (lp_ctx->Globals.param_opt != NULL) { @@ -2509,22 +2515,20 @@ bool lp_load(void) Return the max number of services. ***************************************************************************/ -int lp_numservices(void) +int lp_numservices(struct loadparm_context *lp_ctx) { - return loadparm.iNumServices; + return lp_ctx->iNumServices; } /*************************************************************************** Display the contents of the services array in human-readable form. ***************************************************************************/ -void lp_dump(FILE *f, bool show_defaults, int maxtoprint) +void lp_dump(struct loadparm_context *lp_ctx, FILE *f, bool show_defaults, + int maxtoprint) { - struct loadparm_context *lp_ctx; int iService; - lp_ctx = &loadparm; - if (show_defaults) defaults_saved = false; @@ -2549,28 +2553,30 @@ void lp_dump_one(FILE *f, bool show_defaults, struct loadparm_service *service) } } -struct loadparm_service *lp_servicebynum(int snum) +struct loadparm_service *lp_servicebynum(struct loadparm_context *lp_ctx, + int snum) { - return loadparm.ServicePtrs[snum]; + return lp_ctx->ServicePtrs[snum]; } -struct loadparm_service *lp_service(const char *service_name) +struct loadparm_service *lp_service(struct loadparm_context *lp_ctx, + const char *service_name) { int iService; char *serviceName; - for (iService = loadparm.iNumServices - 1; iService >= 0; iService--) { - if (loadparm.ServicePtrs[iService] && - loadparm.ServicePtrs[iService]->szService) { + for (iService = lp_ctx->iNumServices - 1; iService >= 0; iService--) { + if (lp_ctx->ServicePtrs[iService] && + lp_ctx->ServicePtrs[iService]->szService) { /* * The substitution here is used to support %U is * service names */ serviceName = standard_sub_basic( - loadparm.ServicePtrs[iService], - loadparm.ServicePtrs[iService]->szService); + lp_ctx->ServicePtrs[iService], + lp_ctx->ServicePtrs[iService]->szService); if (strequal(serviceName, service_name)) - return loadparm.ServicePtrs[iService]; + return lp_ctx->ServicePtrs[iService]; } } diff --git a/source4/param/loadparm.h b/source4/param/loadparm.h index d74f70aef0..7f8b375e9a 100644 --- a/source4/param/loadparm.h +++ b/source4/param/loadparm.h @@ -80,4 +80,3 @@ struct parm_struct { #define HOMES_NAME "homes" #endif - diff --git a/source4/param/param.h b/source4/param/param.h index 22c32cb28f..caa5a763bb 100644 --- a/source4/param/param.h +++ b/source4/param/param.h @@ -62,4 +62,6 @@ struct loadparm_service; #include "param/proto.h" +extern struct loadparm_context *global_loadparm; + #endif /* _PARAM_H */ diff --git a/source4/param/share_classic.c b/source4/param/share_classic.c index df86e2be4f..c9786d06bd 100644 --- a/source4/param/share_classic.c +++ b/source4/param/share_classic.c @@ -268,7 +268,7 @@ static NTSTATUS sclassic_list_all(TALLOC_CTX *mem_ctx, int num_services; const char **n; - num_services = lp_numservices(); + num_services = lp_numservices(global_loadparm); n = talloc_array(mem_ctx, const char *, num_services); if (!n) { @@ -277,7 +277,7 @@ static NTSTATUS sclassic_list_all(TALLOC_CTX *mem_ctx, } for (i = 0; i < num_services; i++) { - n[i] = talloc_strdup(n, lp_servicename(lp_servicebynum(i))); + n[i] = talloc_strdup(n, lp_servicename(lp_servicebynum(global_loadparm, i))); if (!n[i]) { DEBUG(0,("ERROR: Out of memory!\n")); talloc_free(n); @@ -299,7 +299,7 @@ static NTSTATUS sclassic_get_config(TALLOC_CTX *mem_ctx, struct share_config *s; struct loadparm_service *service; - service = lp_service(name); + service = lp_service(global_loadparm, name); if (service == NULL) { return NT_STATUS_OBJECT_NAME_NOT_FOUND; diff --git a/source4/scripting/ejs/smbcalls_config.c b/source4/scripting/ejs/smbcalls_config.c index 7161ca49ad..8f1339db34 100644 --- a/source4/scripting/ejs/smbcalls_config.c +++ b/source4/scripting/ejs/smbcalls_config.c @@ -36,8 +36,8 @@ static int ejs_lpServices(MprVarHandle eid, int argc, char **argv) const char **list = NULL; if (argc != 0) return -1; - for (i=0;i<lp_numservices();i++) { - list = str_list_add(list, lp_servicename(lp_servicebynum(i))); + for (i=0;i<lp_numservices(global_loadparm);i++) { + list = str_list_add(list, lp_servicename(lp_servicebynum(global_loadparm, i))); } talloc_steal(mprMemCtx(), list); mpr_Return(eid, mprList("services", list)); @@ -89,7 +89,7 @@ static int ejs_lpGet(MprVarHandle eid, int argc, char **argv) if (argc == 2) { struct loadparm_service *service; /* its a share parameter */ - service = lp_service(argv[0]); + service = lp_service(global_loadparm, argv[0]); if (service == NULL) { mpr_Return(eid, mprCreateUndefinedVar()); return 0; diff --git a/source4/utils/testparm.c b/source4/utils/testparm.c index d884adc7f6..160cf1cc0a 100644 --- a/source4/utils/testparm.c +++ b/source4/utils/testparm.c @@ -150,10 +150,10 @@ static int do_global_checks(void) ret = do_global_checks(); - for (s=0;s<lp_numservices();s++) { - struct loadparm_service *service = lp_servicebynum(s); + for (s=0;s<lp_numservices(global_loadparm);s++) { + struct loadparm_service *service = lp_servicebynum(global_loadparm, s); if (service != NULL) - if (strlen(lp_servicename(lp_servicebynum(s))) > 12) { + if (strlen(lp_servicename(lp_servicebynum(global_loadparm, s))) > 12) { fprintf(stderr, "WARNING: You have some share names that are longer than 12 characters.\n" ); fprintf(stderr, "These may not be accessible to some older clients.\n" ); fprintf(stderr, "(Eg. Windows9x, WindowsMe, and not listed in smbclient in Samba 3.0.)\n" ); @@ -161,8 +161,8 @@ static int do_global_checks(void) } } - for (s=0;s<lp_numservices();s++) { - struct loadparm_service *service = lp_servicebynum(s); + for (s=0;s<lp_numservices(global_loadparm);s++) { + struct loadparm_service *service = lp_servicebynum(global_loadparm, s); if (service != NULL) { const char **deny_list = lp_hostsdeny(service); const char **allow_list = lp_hostsallow(service); @@ -204,7 +204,7 @@ static int do_global_checks(void) section_name = GLOBAL_NAME; service = NULL; } else if ((!strwicmp(section_name, GLOBAL_NAME)) == 0 && - (service=lp_service(section_name)) == NULL) { + (service=lp_service(global_loadparm, section_name)) == NULL) { fprintf(stderr,"Unknown section %s\n", section_name); return(1); @@ -215,15 +215,15 @@ static int do_global_checks(void) ret = !lp_dump_a_parameter(s, parameter_name, stdout, (service == NULL)); } } else { - lp_dump(stdout, show_defaults, lp_numservices()); + lp_dump(global_loadparm, stdout, show_defaults, lp_numservices(global_loadparm)); } return(ret); } if(cname && caddr){ /* this is totally ugly, a real `quick' hack */ - for (s=0;s<lp_numservices();s++) { - struct loadparm_service *service = lp_servicebynum(s); + for (s=0;s<lp_numservices(global_loadparm);s++) { + struct loadparm_service *service = lp_servicebynum(global_loadparm, s); if (service != NULL) { if (allow_access(NULL, lp_hostsdeny(NULL), lp_hostsallow(NULL), cname, caddr) && allow_access(NULL, lp_hostsdeny(service), lp_hostsallow(service), cname, caddr)) { |