Removed unrequired code

[libplp.git] / src / config.c

diff --git a/src/config.c b/src/config.c
index c099d97fbe022b89c12b0549667210427b38c987..4e51b43ca78db17ddc946968775567b2b6df2adf 100644 (file)
--- a/src/config.c
+++ b/src/config.c
@@ -26,12 +26,15 @@ static int uci_create_named_section(struct uci_context *ctx, const char *conf_fi
                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);
+               if (cmd_data != NULL) {
+                       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);
+                               }
                        }
+                       free(cmd_data);
                }
        }
        return ret_val;
@@ -47,18 +50,15 @@ bool set_config_value(const char *conf_dir_name,
        struct uci_package      *pkg;
        struct uci_section      *sct;
        struct uci_section      *tmp_sct;
-       struct uci_option       *opt;
        int                     err_flg;
        char                    *fname;
        int                     b_count;
-       bool                    save;
        struct uci_element      *elem;
        struct uci_ptr          ptr;
        FILE                    *fp;
        bool                    ret_val;
 
        ret_val = false;
-       save    = false;
        if ((conf_dir_name != NULL) &&
            (conf_file_name != NULL) &&
            (section_type != NULL) &&
@@ -106,45 +106,39 @@ bool set_config_value(const char *conf_dir_name,
                                                }
                                        }
                                        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;
+                                               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) {
-                                                               ptr.value       = strdup(value);
-                                                               uci_set(ctx, &ptr);
-                                                               save    = true;
-                                                       }
+                                               if (uci_lookup_ptr(ctx, &ptr, NULL, false) == UCI_OK) {
+                                                       ptr.value       = value;
+                                                       uci_set(ctx, &ptr);
+                                                       uci_save(ctx, pkg);
+                                                       ret_val         = true;
+                                                       log_debug("Created configuration section %s to configuration file: %s\n", section_name, fname);
                                                }
                                                else {
-                                                       opt     = uci_alloc_option(sct, key, value);
-                                                       save    = true;
-                                               }
-                                               if (save == true) {
-                                                       uci_save(ctx, pkg);
+                                                       log_error("Could not write to configuration file: %s\n. Could not look-up pointer for package %s section %s.\n", fname, pkg->e.name, sct->e.name);
                                                }
                                                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);
+                                               log_error("Could not write to configuration file: %s. Could not create section %s.\n", fname, section_name);
                                        }
                                }
                                else {
-                                       log_error("Could not write to configuration file: %s\n. File does not exist or is not writable.", fname);
+                                       log_error("Could not write to configuration file: %s. File does not exist or is not writable\n.", fname);
                                }
                        }
                        free(fname);
                }
                else {
-                       log_error("Could not change config value, out of memory");
+                       log_error("Could not change config value, out of memory\n");
                }
        }
        else {
-               log_error("Could not change config value, invalid parameters");
+               log_error("Could not change config value, invalid parameters\n");
        }
        return ret_val;
 }