s3-param use common struct parmlist_entry to hold parametric options

This will allow struct loadparm_service to be shared in future.

Andrew Bartlett

diff --git a/lib/param/loadparm.h b/lib/param/loadparm.h
index 2627727f3a7a092ea89f404370b681af4696b8e1..d36f5a0f6e44942434eb57556318eaffde5fda42 100644 (file)
--- a/lib/param/loadparm.h
+++ b/lib/param/loadparm.h
@@ -27,7 +27,8 @@
    along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-/* the following are used by loadparm for option lists */
+#include "../lib/util/parmlist.h"
+
 /* the following are used by loadparm for option lists */
 typedef enum {
        P_BOOL,P_BOOLREV,P_CHAR,P_INTEGER,P_OCTAL,P_LIST,

diff --git a/lib/util/parmlist.h b/lib/util/parmlist.h
index b320afee4701265faaa4d6f8cde2e2235100f988..9bc4f36126866efab37d5979310bc796779dcd37 100644 (file)
--- a/lib/util/parmlist.h
+++ b/lib/util/parmlist.h
@@ -24,6 +24,7 @@ struct parmlist_entry {
        struct parmlist_entry *prev, *next;
        char *key;
        char *value;
+       char **list; /* For the source3 parametric options, to save the parsed list */
        int priority;
 };
 

diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index 361597d9158c8164d0077777974f0e78e231f168..a41e9f7e32c2e9bf063069e43bbba6f0a073a333 100644 (file)
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -106,16 +106,8 @@ static int config_backend = CONFIG_BACKEND_FILE;
 
 static bool defaults_saved = false;
 
-struct param_opt_struct {
-       struct param_opt_struct *prev, *next;
-       char *key;
-       char *value;
-       char **list;
-       unsigned flags;
-};
-
 #define LOADPARM_EXTRA_GLOBALS \
-       struct param_opt_struct *param_opt; \
+       struct parmlist_entry *param_opt;                               \
        char *szRealm;                                                  \
        char *szLogLevel;                                               \
        int iminreceivefile;                                            \
@@ -144,7 +136,7 @@ struct param_opt_struct {
        char *szInclude;                                                \
        bool bAvailable;                                                \
        bool bWidelinks;                                                \
-       struct param_opt_struct *param_opt;                             \
+       struct parmlist_entry *param_opt;                               \
        struct bitmap *copymap;                                         \
        char dummy[3];          /* for alignment */
 
@@ -5563,7 +5555,7 @@ static bool do_section(const char *pszSectionName, void *userdata);
 static void init_copymap(struct loadparm_service *pservice);
 static bool hash_a_service(const char *name, int number);
 static void free_service_byindex(int iService);
-static void free_param_opts(struct param_opt_struct **popts);
+static void free_param_opts(struct parmlist_entry **popts);
 static void show_parameter(int parmIndex);
 static bool is_synonym_of(int parm1, int parm2, bool *inverse);
 
@@ -5572,12 +5564,12 @@ static bool is_synonym_of(int parm1, int parm2, bool *inverse);
  * pointer to parametrical option value if it exists or NULL otherwise. Actual
  * parametrical functions are quite simple
  */
-static struct param_opt_struct *get_parametrics_by_service(struct loadparm_service *service, const char *type,
+static struct parmlist_entry *get_parametrics_by_service(struct loadparm_service *service, const char *type,
                                                           const char *option)
 {
        bool global_section = false;
        char* param_key;
-        struct param_opt_struct *data;
+        struct parmlist_entry *data;
 
        if (service == NULL) {
                data = Globals.param_opt;
@@ -5622,11 +5614,9 @@ static struct param_opt_struct *get_parametrics_by_service(struct loadparm_servi
  * pointer to parametrical option value if it exists or NULL otherwise. Actual
  * parametrical functions are quite simple
  */
-static struct param_opt_struct *get_parametrics(int snum, const char *type,
+static struct parmlist_entry *get_parametrics(int snum, const char *type,
                                                const char *option)
 {
-        struct param_opt_struct *data;
-
        if (snum >= iNumServices) return NULL;
 
        if (snum < 0) {
@@ -5716,7 +5706,7 @@ static int lp_enum(const char *s,const struct enum_list *_enum)
 /* the returned value is talloced on the talloc_tos() */
 char *lp_parm_talloc_string(int snum, const char *type, const char *option, const char *def)
 {
-       struct param_opt_struct *data = get_parametrics(snum, type, option);
+       struct parmlist_entry *data = get_parametrics(snum, type, option);
 
        if (data == NULL||data->value==NULL) {
                if (def) {
@@ -5733,7 +5723,7 @@ char *lp_parm_talloc_string(int snum, const char *type, const char *option, cons
 /* Parametric option has following syntax: 'Type: option = value' */
 const char *lp_parm_const_string(int snum, const char *type, const char *option, const char *def)
 {
-       struct param_opt_struct *data = get_parametrics(snum, type, option);
+       struct parmlist_entry *data = get_parametrics(snum, type, option);
 
        if (data == NULL||data->value==NULL)
                return def;
@@ -5743,7 +5733,7 @@ const char *lp_parm_const_string(int snum, const char *type, const char *option,
 
 const char *lp_parm_const_string_service(struct loadparm_service *service, const char *type, const char *option)
 {
-       struct param_opt_struct *data = get_parametrics_by_service(service, type, option);
+       struct parmlist_entry *data = get_parametrics_by_service(service, type, option);
 
        if (data == NULL||data->value==NULL)
                return NULL;
@@ -5757,7 +5747,7 @@ const char *lp_parm_const_string_service(struct loadparm_service *service, const
 
 const char **lp_parm_string_list(int snum, const char *type, const char *option, const char **def)
 {
-       struct param_opt_struct *data = get_parametrics(snum, type, option);
+       struct parmlist_entry *data = get_parametrics(snum, type, option);
 
        if (data == NULL||data->value==NULL)
                return (const char **)def;
@@ -5774,7 +5764,7 @@ const char **lp_parm_string_list(int snum, const char *type, const char *option,
 
 int lp_parm_int(int snum, const char *type, const char *option, int def)
 {
-       struct param_opt_struct *data = get_parametrics(snum, type, option);
+       struct parmlist_entry *data = get_parametrics(snum, type, option);
 
        if (data && data->value && *data->value)
                return lp_int(data->value);
@@ -5787,7 +5777,7 @@ int lp_parm_int(int snum, const char *type, const char *option, int def)
 
 unsigned long lp_parm_ulong(int snum, const char *type, const char *option, unsigned long def)
 {
-       struct param_opt_struct *data = get_parametrics(snum, type, option);
+       struct parmlist_entry *data = get_parametrics(snum, type, option);
 
        if (data && data->value && *data->value)
                return lp_ulong(data->value);
@@ -5800,7 +5790,7 @@ unsigned long lp_parm_ulong(int snum, const char *type, const char *option, unsi
 
 bool lp_parm_bool(int snum, const char *type, const char *option, bool def)
 {
-       struct param_opt_struct *data = get_parametrics(snum, type, option);
+       struct parmlist_entry *data = get_parametrics(snum, type, option);
 
        if (data && data->value && *data->value)
                return lp_bool(data->value);
@@ -5814,7 +5804,7 @@ bool lp_parm_bool(int snum, const char *type, const char *option, bool def)
 int lp_parm_enum(int snum, const char *type, const char *option,
                 const struct enum_list *_enum, int def)
 {
-       struct param_opt_struct *data = get_parametrics(snum, type, option);
+       struct parmlist_entry *data = get_parametrics(snum, type, option);
 
        if (data && data->value && *data->value && _enum)
                return lp_enum(data->value, _enum);
@@ -5840,9 +5830,9 @@ static void init_service(struct loadparm_service *pservice)
  * then this whole functions reduces to a TALLOC_FREE().
  */
 
-static void free_param_opts(struct param_opt_struct **popts)
+static void free_param_opts(struct parmlist_entry **popts)
 {
-       struct param_opt_struct *opt, *next_opt;
+       struct parmlist_entry *opt, *next_opt;
 
        if (popts == NULL) {
                return;
@@ -6572,15 +6562,15 @@ struct loadparm_service *lp_service(const char *pszServiceName)
 ***************************************************************************/
 
 /**
- * Add a parametric option to a param_opt_struct,
+ * Add a parametric option to a parmlist_entry,
  * replacing old value, if already present.
  */
-static void set_param_opt(struct param_opt_struct **opt_list,
+static void set_param_opt(struct parmlist_entry **opt_list,
                          const char *opt_name,
                          const char *opt_value,
-                         unsigned flags)
+                         unsigned priority)
 {
-       struct param_opt_struct *new_opt, *opt;
+       struct parmlist_entry *new_opt, *opt;
        bool not_added;
 
        if (opt_list == NULL) {
@@ -6594,8 +6584,8 @@ static void set_param_opt(struct param_opt_struct **opt_list,
        while (opt) {
                /* If we already have same option, override it */
                if (strwicmp(opt->key, opt_name) == 0) {
-                       if ((opt->flags & FLAG_CMDLINE) &&
-                           !(flags & FLAG_CMDLINE)) {
+                       if ((opt->priority & FLAG_CMDLINE) &&
+                           !(priority & FLAG_CMDLINE)) {
                                /* it's been marked as not to be
                                   overridden */
                                return;
@@ -6603,18 +6593,18 @@ static void set_param_opt(struct param_opt_struct **opt_list,
                        string_free(&opt->value);
                        TALLOC_FREE(opt->list);
                        opt->value = SMB_STRDUP(opt_value);
-                       opt->flags = flags;
+                       opt->priority = priority;
                        not_added = false;
                        break;
                }
                opt = opt->next;
        }
        if (not_added) {
-           new_opt = SMB_XMALLOC_P(struct param_opt_struct);
+           new_opt = SMB_XMALLOC_P(struct parmlist_entry);
            new_opt->key = SMB_STRDUP(opt_name);
            new_opt->value = SMB_STRDUP(opt_value);
            new_opt->list = NULL;
-           new_opt->flags = flags;
+           new_opt->priority = priority;
            DLIST_ADD(*opt_list, new_opt);
        }
 }
@@ -6624,7 +6614,7 @@ static void copy_service(struct loadparm_service *pserviceDest, struct loadparm_
 {
        int i;
        bool bcopyall = (pcopymapDest == NULL);
-       struct param_opt_struct *data;
+       struct parmlist_entry *data;
 
        for (i = 0; parm_table[i].label; i++)
                if (parm_table[i].p_class == P_LOCAL &&
@@ -6681,7 +6671,7 @@ static void copy_service(struct loadparm_service *pserviceDest, struct loadparm_
 
        data = pserviceSource->param_opt;
        while (data) {
-               set_param_opt(&pserviceDest->param_opt, data->key, data->value, data->flags);
+               set_param_opt(&pserviceDest->param_opt, data->key, data->value, data->priority);
                data = data->next;
        }
 }
@@ -7432,7 +7422,7 @@ bool lp_do_parameter(int snum, const char *pszParmName, const char *pszParmValue
 {
        int parmnum, i;
        void *parm_ptr = NULL;  /* where we are going to store the result */
-       struct param_opt_struct **opt_list;
+       struct parmlist_entry **opt_list;
 
        parmnum = map_parameter(pszParmName);
 
@@ -7867,7 +7857,7 @@ Display the contents of the global structure.
 static void dump_globals(FILE *f)
 {
        int i;
-       struct param_opt_struct *data;
+       struct parmlist_entry *data;
 
        fprintf(f, "[global]\n");
 
@@ -7911,7 +7901,7 @@ bool lp_is_default(int snum, struct parm_struct *parm)
 static void dump_a_service(struct loadparm_service *pService, FILE * f)
 {
        int i;
-       struct param_opt_struct *data;
+       struct parmlist_entry *data;
 
        if (pService != &sDefault)
                fprintf(f, "[%s]\n", pService->szService);