onewire_parser.c: Fix compiler warnings about string lengths

[rrdd] / config.c

diff --git a/config.c b/config.c
index 8ac1b63a867fad23e727938afad3fdcdf9792ebc..83242033d6958c329d51c219c92c29c683839a2d 100644 (file)
--- a/config.c
+++ b/config.c
@@ -129,11 +129,11 @@ static int parse_images(config_setting_t *list, struct rrd_database *db)
                db->images[i]->width            = width;
                db->images[i]->height           = height;
                strncpy(db->images[i]->timestart, timestart,
-                       sizeof(db->images[i]->timestart));
+                       sizeof(db->images[i]->timestart) - 1);
                strncpy(db->images[i]->timeend, timeend,
-                       sizeof(db->images[i]->timeend));
+                       sizeof(db->images[i]->timeend) - 1);
                strncpy(db->images[i]->imageformat, imageformat,
-                       sizeof(db->images[i]->imageformat));
+                       sizeof(db->images[i]->imageformat) - 1);
                db->images[i]->options          = options;
                db->images[i]->text_lead        = text_lead;
                db->images[i]->text             = text;
@@ -243,10 +243,9 @@ static int parse_database(config_setting_t *rrd, struct rrd_database *db)
        pr_info("parsing database %s\n", name);
        db->name = strdup(name);
 
-       if (!config_setting_lookup_string(rrd, "filename", &filename)) {
+       if (!config_setting_lookup_string(rrd, "filename", &filename))
                pr_err("Database %s does not contain filename\n", db->name);
-               return -1;
-       }
+
        db->filename = strdup(filename);
 
        if (!config_setting_lookup_int(rrd, "interval", &db->interval)) {
@@ -278,7 +277,7 @@ static int parse_database(config_setting_t *rrd, struct rrd_database *db)
        str_list = config_setting_get_member(rrd, "post_draw_cmd");
        if (str_list) {
                read_strings_from_list(str_list, &post_draw_cmd);
-               db->pre_draw_cmd = (char *const *)post_draw_cmd;
+               db->post_draw_cmd = (char *const *)post_draw_cmd;
        }
 
        config_setting_lookup_string(rrd, "logfile", &logfile);