Adjust codebase for iniparser upgrade (4.x) 79/222079/1
authorKarol Lewandowski <k.lewandowsk@samsung.com>
Thu, 9 Jan 2020 10:22:00 +0000 (11:22 +0100)
committerKarol Lewandowski <k.lewandowsk@samsung.com>
Thu, 9 Jan 2020 10:22:00 +0000 (11:22 +0100)
Change-Id: I3d8e613b8d1cccbbbf84b7dceac5abde57578fe6

src/dump_systemstate/extras.c
src/shared/config.c

index 90625a5..2e6939c 100644 (file)
@@ -62,7 +62,7 @@ static const size_t MAX_INI_KEY_LEN = 5;
 
 struct extra_dump_item {
        // not separate named fields, for convenient iteration
-       char *fields[COUNT_INI_FIELDS];
+       const char *fields[COUNT_INI_FIELDS];
 };
 
 int handle_extra_program(int out_fd, struct extra_dump_item *item)
@@ -70,10 +70,10 @@ int handle_extra_program(int out_fd, struct extra_dump_item *item)
        assert(out_fd >= 0);
        assert(item);
 
-       char *const title = item->fields[INI_FIELD_TITLE];
-       char *const path  = item->fields[INI_FIELD_PATH];
-       char *const args  = item->fields[INI_FIELD_ARGS] ?: "";
-       char *const env   = item->fields[INI_FIELD_ENV] ?: "";
+       const char *const title = item->fields[INI_FIELD_TITLE];
+       const char *const path  = item->fields[INI_FIELD_PATH];
+       const char *const args  = item->fields[INI_FIELD_ARGS] ?: "";
+       const char *const env   = item->fields[INI_FIELD_ENV] ?: "";
 
        if (!title || !path) {
                fprintf_fd(out_fd, "\nNo title or path in extra program config");
@@ -109,8 +109,8 @@ int handle_extra_file(int out_fd, struct extra_dump_item *item)
        assert(out_fd >= 0);
        assert(item);
 
-       char *const title = item->fields[INI_FIELD_TITLE];
-       char *const path  = item->fields[INI_FIELD_PATH];
+       const char *const title = item->fields[INI_FIELD_TITLE];
+       const char *const path  = item->fields[INI_FIELD_PATH];
        if (!title || !path) {
                fprintf_fd(out_fd, "\nNo title or path in extra file config");
                return EXIT_CONFERR;
@@ -127,7 +127,7 @@ int handle_extra_file(int out_fd, struct extra_dump_item *item)
 
 typedef int (*handle_ini_section_t)(int out_fd, struct extra_dump_item *);
 
-static int handle_ini_Nth_section(int out_fd, dictionary *ini, int n, handle_ini_section_t handle_ini_section)
+static int handle_ini_Nth_section(int out_fd, const dictionary *ini, int n, handle_ini_section_t handle_ini_section)
 {
        assert(out_fd >= 0);
        assert(ini);
@@ -135,7 +135,7 @@ static int handle_ini_Nth_section(int out_fd, dictionary *ini, int n, handle_ini
        assert(n < iniparser_getnsec(ini));
        assert(handle_ini_section);
 
-       char *const secname = iniparser_getsecname(ini, n);
+       const char *const secname = iniparser_getsecname(ini, n);
        assert(secname); // can only be NULL if `ini` is NULL or `n` is outta bounds
 
        const size_t secname_len = strlen(secname);
index d389187..88df6a2 100644 (file)
@@ -53,11 +53,11 @@ enum ReportType report_type_from_str(const char *report_type_str)
        return REP_TYPE_INVALID;
 }
 
-static bool config_load_from_dict(config_t *c, dictionary *ini)
+static bool config_load_from_dict(config_t *c, const dictionary *ini)
 {
        assert(c);
 
-       char *str = iniparser_getstring(ini, CRASH_SECTION ":CrashRootPath", NULL);
+       const char *str = iniparser_getstring(ini, CRASH_SECTION ":CrashRootPath", NULL);
        if (str) {
                char *crash_root_path = strdup(str);
                if (!crash_root_path) {