X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Ftypes_list.c;h=7600c346ecc1e8c901460a029c2ead4c167e5576;hb=92445ff3363441d0f515de4a3ab92a504cfc0366;hp=002761ce3e7a76568df04612058d1492825595d2;hpb=2b6176cab4f092354177473bbc74c5cdc2eaa2ec;p=collectd.git diff --git a/src/types_list.c b/src/types_list.c index 002761ce..7600c346 100644 --- a/src/types_list.c +++ b/src/types_list.c @@ -34,7 +34,7 @@ static int parse_ds (data_source_t *dsrc, char *buf, size_t buf_len) if (buf_len < 11) { - ERROR ("parse_ds: (buf_len = %u) < 11", buf_len); + ERROR ("parse_ds: (buf_len = %zu) < 11", buf_len); return (-1); } @@ -62,8 +62,7 @@ static int parse_ds (data_source_t *dsrc, char *buf, size_t buf_len) return (-1); } - strncpy (dsrc->name, fields[0], sizeof (dsrc->name)); - dsrc->name[sizeof (dsrc->name) - 1] = '\0'; + sstrncpy (dsrc->name, fields[0], sizeof (dsrc->name)); if (strcasecmp (fields[1], "GAUGE") == 0) dsrc->type = DS_TYPE_GAUGE; @@ -85,13 +84,10 @@ static int parse_ds (data_source_t *dsrc, char *buf, size_t buf_len) else dsrc->max = atof (fields[3]); - DEBUG ("parse_ds: dsrc = {%s, %i, %lf, %lf};", - dsrc->name, dsrc->type, dsrc->min, dsrc->max); - return (0); } /* int parse_ds */ -static void parse_line (char *buf, size_t buf_len) +static void parse_line (char *buf) { char *fields[64]; size_t fields_num; @@ -108,8 +104,7 @@ static void parse_line (char *buf, size_t buf_len) memset (ds, '\0', sizeof (data_set_t)); - strncpy (ds->type, fields[0], sizeof (ds->type)); - ds->type[sizeof (ds->type) - 1] = '\0'; + sstrncpy (ds->type, fields[0], sizeof (ds->type)); ds->ds_num = fields_num - 1; ds->ds = (data_source_t *) calloc (ds->ds_num, sizeof (data_source_t)); @@ -125,9 +120,6 @@ static void parse_line (char *buf, size_t buf_len) return; } - DEBUG ("parse_line: ds = {%s, %i, %p};", - ds->type, ds->ds_num, (void *) ds->ds); - plugin_register_data_set (ds); sfree (ds->ds); @@ -165,21 +157,16 @@ static void parse_file (FILE *fh) if (buf_len == 0) continue; - parse_line (buf, buf_len); + parse_line (buf); } /* while (fgets) */ } /* void parse_file */ -int read_types_list (void) +int read_types_list (const char *file) { - const char *file; FILE *fh; - file = global_option_get ("TypesDB"); if (file == NULL) - { - ERROR ("global_option_get (\"TypesDB\") returned NULL."); return (-1); - } fh = fopen (file, "r"); if (fh == NULL)