X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Ftypes_list.c;h=887b43d87c420d76a823a64452fe788893ff1bd1;hb=21ba59f8197dba0a72dbf5186bc0a5d7e5074496;hp=3be792d5ca11bbd741f9b599a517c9cfea7da9ac;hpb=3f2f61c7d74d9204405a4b0c21390451c98665a7;p=collectd.git diff --git a/src/types_list.c b/src/types_list.c index 3be792d5..887b43d8 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,16 +62,19 @@ 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; else if (strcasecmp (fields[1], "COUNTER") == 0) dsrc->type = DS_TYPE_COUNTER; + else if (strcasecmp (fields[1], "DERIVE") == 0) + dsrc->type = DS_TYPE_DERIVE; + else if (strcasecmp (fields[1], "ABSOLUTE") == 0) + dsrc->type = DS_TYPE_ABSOLUTE; else { - ERROR ("(fields[1] = %s) != (GAUGE || COUNTER)", fields[1]); + ERROR ("(fields[1] = %s) != (GAUGE || COUNTER || DERIVE || ABSOLUTE)", fields[1]); return (-1); } @@ -99,14 +102,17 @@ static void parse_line (char *buf) if (fields_num < 2) return; + /* Ignore lines which begin with a hash sign. */ + if (fields[0][0] == '#') + return; + ds = (data_set_t *) malloc (sizeof (data_set_t)); if (ds == NULL) return; 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)); @@ -153,7 +159,7 @@ static void parse_file (FILE *fh) continue; while ((buf_len > 0) && ((buf[buf_len - 1] == '\n') - || (buf[buf_len - 1] == '\n'))) + || (buf[buf_len - 1] == '\r'))) buf[--buf_len] = '\0'; if (buf_len == 0) @@ -174,7 +180,9 @@ int read_types_list (const char *file) if (fh == NULL) { char errbuf[1024]; - ERROR ("open (%s) failed: %s", + fprintf (stderr, "Failed to open types database `%s': %s.\n", + file, sstrerror (errno, errbuf, sizeof (errbuf))); + ERROR ("Failed to open types database `%s': %s", file, sstrerror (errno, errbuf, sizeof (errbuf))); return (-1); }