named sections can be done also without adding new api function

[libplp.git] / src / config.c

diff --git a/src/config.c b/src/config.c
index 74bfd8bd15a07fa3605570a1cd588da46eb7f99e..c099d97fbe022b89c12b0549667210427b38c987 100644 (file)
--- a/src/config.c
+++ b/src/config.c
@@ -7,6 +7,36 @@
 #include "log.h"
 #include "config.h"
 
+static int uci_create_named_section(struct uci_context *ctx, const char *conf_file_name, const char *section_type, const char *section_name)
+{
+       struct uci_ptr          ptr;
+       int                     ret_val;
+       char                    *cmd_data;
+       int                     len;
+
+       ret_val = -1;
+       if ((ctx != NULL) &&
+           (conf_file_name != NULL) &&
+           (section_type != NULL) &&
+           (section_name != NULL)) {
+               len             = strlen(conf_file_name);
+               len             = len + 1;
+               len             = len + strlen(section_type);
+               len             = len + 1;
+               len             = len + strlen(section_name);
+               len             = len + 1;
+               cmd_data        = malloc(len);
+               snprintf(cmd_data, len, "%s.%s=%s", conf_file_name, section_name, section_type);
+               if (uci_lookup_ptr(ctx, &ptr, cmd_data, true) == UCI_OK) {
+                       ret_val = uci_set(ctx, &ptr);
+                       if (ret_val == UCI_OK) {
+                               //ret_val       = uci_save(ctx, ptr.p);
+                       }
+               }
+       }
+       return ret_val;
+}
+
 bool set_config_value(const char *conf_dir_name,
                        const char *conf_file_name,
                        const char *section_type,
@@ -22,7 +52,9 @@ bool set_config_value(const char *conf_dir_name,
        char                    *fname;
        int                     b_count;
        bool                    save;
-       struct uci_element      *e;
+       struct uci_element      *elem;
+       struct uci_ptr          ptr;
+       FILE                    *fp;
        bool                    ret_val;
 
        ret_val = false;
@@ -45,53 +77,65 @@ bool set_config_value(const char *conf_dir_name,
                                uci_set_confdir(ctx, conf_dir_name);
                                if (access(fname, W_OK) != 0) {
                                        if (access(fname, F_OK) != 0) {
-                                               FILE *fp;
                                                fp      = fopen(fname, "w+");
                                                fclose(fp);
                                        }
                                }
-                               err_flg = uci_load(ctx, fname, &pkg);
-                               uci_foreach_element(&pkg->sections, e) {
-                                       tmp_sct = uci_to_section(e);
-                                       if (strcmp(tmp_sct->type, section_type) == 0) {
-                                               sct     = tmp_sct;
-                                               break;
+                               if (access(fname, W_OK) == 0) {
+                                       err_flg = uci_load(ctx, fname, &pkg);
+                                       uci_foreach_element(&pkg->sections, elem) {
+                                               tmp_sct = uci_to_section(elem);
+                                               if (strcmp(tmp_sct->type, section_type) == 0) {
+                                                       sct     = tmp_sct;
+                                                       break;
+                                               }
                                        }
-                               }
-                               //sct   = uci_lookup_section(ctx, pkg, "service");
-                               if (sct != NULL) {
-                                       log_debug("section found\n");
-                               }
-                               else {
-                                       log_debug("adding new section\n");
-                                       err_flg = uci_add_named_section(ctx, pkg, section_type, section_name, &sct);
-                                       log_debug("err_flg: %d, section name: %s\n", err_flg, sct->e.name);
-                               }
-                               opt     = uci_lookup_option(ctx, sct, key);
-                               if (opt != NULL) {
-                                       struct uci_ptr ptr;
-                                       memset(&ptr, 0, sizeof(ptr));
-                                       ptr.package     = pkg->e.name;
-                                       ptr.section     = sct->e.name;
-                                       ptr.option      = key;
+                                       if (sct == NULL) {
+                                               log_debug("Creating configuration section %s to configuration file: %s\n", section_name, fname);
+                                               //err_flg       = uci_add_named_section(ctx, pkg, section_type, section_name, &sct);
+                                               //err_flg       = uci_add_section(ctx, pkg, section_name, &sct);
+                                               err_flg = uci_create_named_section(ctx, conf_file_name, section_type, section_name);
+                                               if (err_flg == UCI_OK) {
+                                                       uci_foreach_element(&pkg->sections, elem) {
+                                                               tmp_sct = uci_to_section(elem);
+                                                               if (strcmp(tmp_sct->type, section_type) == 0) {
+                                                                       sct     = tmp_sct;
+                                                                       break;
+                                                               }
+                                                       }
+                                               }
+                                       }
+                                       if (err_flg == 0) {
+                                               opt     = uci_lookup_option(ctx, sct, key);
+                                               if (opt != NULL) {
+                                                       memset(&ptr, 0, sizeof(ptr));
+                                                       ptr.package     = pkg->e.name;
+                                                       ptr.section     = sct->e.name;
+                                                       ptr.option      = key;
 
-                                       if (uci_lookup_ptr(ctx, &ptr, NULL, false) == UCI_OK) {
-                                               log_debug("trying to change option value\n");
-                                               ptr.value       = strdup(value);
-                                               uci_set(ctx, &ptr);
-                                               save            = true;
+                                                       if (uci_lookup_ptr(ctx, &ptr, NULL, false) == UCI_OK) {
+                                                               ptr.value       = strdup(value);
+                                                               uci_set(ctx, &ptr);
+                                                               save    = true;
+                                                       }
+                                               }
+                                               else {
+                                                       opt     = uci_alloc_option(sct, key, value);
+                                                       save    = true;
+                                               }
+                                               if (save == true) {
+                                                       uci_save(ctx, pkg);
+                                               }
+                                               uci_free_context(ctx);
+                                               ret_val = true;
+                                       }
+                                       else {
+                                               log_error("Could not write to configuration file: %s\n. Could not create section %s.", fname, section_name);
                                        }
                                }
                                else {
-                                       opt     = uci_alloc_option(sct, key, value);
-                                       save    = true;
-                               }
-                               if (save == true) {
-                                       log_debug("saving config file: %s\n", fname);
-                                       uci_save(ctx, pkg);
+                                       log_error("Could not write to configuration file: %s\n. File does not exist or is not writable.", fname);
                                }
-                               uci_free_context(ctx);
-                               ret_val = true;
                        }
                        free(fname);
                }