X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fmd.c;h=0a015c7b0088f3796ff14a7f2f085cb235407aca;hb=07ba05937aeaedd683656c3912040950dbf4a152;hp=94965e6e42a7d71a96acd3a097735de6b0efd6bc;hpb=2079ee1517e34de372f58e7e2267ad5c71a8a41f;p=collectd.git diff --git a/src/md.c b/src/md.c index 94965e6e..0a015c7b 100644 --- a/src/md.c +++ b/src/md.c @@ -40,37 +40,33 @@ static const char *config_keys[] = {"Device", "IgnoreSelected"}; static int config_keys_num = STATIC_ARRAY_SIZE(config_keys); -static ignorelist_t *ignorelist = NULL; +static ignorelist_t *ignorelist; static int md_config(const char *key, const char *value) { if (ignorelist == NULL) ignorelist = ignorelist_create(/* invert = */ 1); if (ignorelist == NULL) - return (1); + return 1; if (strcasecmp(key, "Device") == 0) { ignorelist_add(ignorelist, value); } else if (strcasecmp(key, "IgnoreSelected") == 0) { ignorelist_set_invert(ignorelist, IS_TRUE(value) ? 0 : 1); } else { - return (-1); + return -1; } - return (0); + return 0; } static void md_submit(const int minor, const char *type_instance, gauge_t value) { - value_t values[1]; value_list_t vl = VALUE_LIST_INIT; - values[0].gauge = value; - - vl.values = values; + vl.values = &(value_t){.gauge = value}; vl.values_len = 1; - sstrncpy(vl.host, hostname_g, sizeof(vl.host)); sstrncpy(vl.plugin, "md", sizeof(vl.plugin)); - ssnprintf(vl.plugin_instance, sizeof(vl.plugin_instance), "%i", minor); + snprintf(vl.plugin_instance, sizeof(vl.plugin_instance), "%i", minor); sstrncpy(vl.type, "md_disks", sizeof(vl.type)); sstrncpy(vl.type_instance, type_instance, sizeof(vl.type_instance)); @@ -78,7 +74,6 @@ static void md_submit(const int minor, const char *type_instance, } /* void md_submit */ static void md_process(const int minor, const char *path) { - char errbuf[1024]; int fd; struct stat st; mdu_array_info_t array; @@ -86,13 +81,12 @@ static void md_process(const int minor, const char *path) { fd = open(path, O_RDONLY); if (fd < 0) { - WARNING("md: open(%s): %s", path, sstrerror(errno, errbuf, sizeof(errbuf))); + WARNING("md: open(%s): %s", path, STRERRNO); return; } if (fstat(fd, &st) < 0) { - WARNING("md: Unable to fstat file descriptor for %s: %s", path, - sstrerror(errno, errbuf, sizeof(errbuf))); + WARNING("md: Unable to fstat file descriptor for %s: %s", path, STRERRNO); close(fd); return; } @@ -113,8 +107,7 @@ static void md_process(const int minor, const char *path) { /* Retrieve md information */ if (ioctl(fd, GET_ARRAY_INFO, &array) < 0) { - WARNING("md: Unable to retrieve array info from %s: %s", path, - sstrerror(errno, errbuf, sizeof(errbuf))); + WARNING("md: Unable to retrieve array info from %s: %s", path, STRERRNO); close(fd); return; } @@ -150,10 +143,8 @@ static int md_read(void) { fh = fopen(PROC_DISKSTATS, "r"); if (fh == NULL) { - char errbuf[1024]; - WARNING("md: Unable to open %s: %s", PROC_DISKSTATS, - sstrerror(errno, errbuf, sizeof(errbuf))); - return (-1); + WARNING("md: Unable to open %s: %s", PROC_DISKSTATS, STRERRNO); + return -1; } /* Iterate md devices */ @@ -184,14 +175,14 @@ static int md_read(void) { * major/minor, but that again can be tricky if the filesystem * with the device file is mounted using the "nodev" option. */ - ssnprintf(path, sizeof(path), "%s/%s", DEV_DIR, name); + snprintf(path, sizeof(path), "%s/%s", DEV_DIR, name); md_process(minor, path); } fclose(fh); - return (0); + return 0; } /* int md_read */ void module_register(void) {