X-Git-Url: https://git.octo.it/?p=collectd.git;a=blobdiff_plain;f=src%2Ffilecount.c;h=5acd47b3339e99d259fce8a9c1c5513e987f2b5a;hp=7842aa610c786719f61762b11ca55ad6d9565941;hb=54619dc85fd308b21ed09a0271e5c7383c7921b9;hpb=3d887357f3c5034bd95731734f4b5c79b0b00ad0 diff --git a/src/filecount.c b/src/filecount.c index 7842aa61..5acd47b3 100644 --- a/src/filecount.c +++ b/src/filecount.c @@ -23,8 +23,8 @@ #include "collectd.h" -#include "common.h" #include "plugin.h" +#include "utils/common/common.h" #include #include @@ -60,10 +60,10 @@ struct fc_directory_conf_s { }; typedef struct fc_directory_conf_s fc_directory_conf_t; -static fc_directory_conf_t **directories = NULL; -static size_t directories_num = 0; +static fc_directory_conf_t **directories; +static size_t directories_num; -void fc_free_dir(fc_directory_conf_t *dir) { +static void fc_free_dir(fc_directory_conf_t *dir) { sfree(dir->path); sfree(dir->plugin_name); sfree(dir->instance); @@ -154,26 +154,6 @@ static int fc_config_add_dir_instance(fc_directory_conf_t *dir, return fc_config_set_instance(dir, ci->values[0].value.string); } /* int fc_config_add_dir_instance */ -static int fc_config_add_dir_name(fc_directory_conf_t *dir, - oconfig_item_t *ci) { - if ((ci->values_num != 1) || (ci->values[0].type != OCONFIG_TYPE_STRING)) { - WARNING("filecount plugin: The `Name' config option needs exactly one " - "string argument."); - return -1; - } - - char *temp = strdup(ci->values[0].value.string); - if (temp == NULL) { - ERROR("filecount plugin: strdup failed."); - return -1; - } - - sfree(dir->name); - dir->name = temp; - - return 0; -} /* int fc_config_add_dir_name */ - static int fc_config_add_dir_mtime(fc_directory_conf_t *dir, oconfig_item_t *ci) { if ((ci->values_num != 1) || ((ci->values[0].type != OCONFIG_TYPE_STRING) && @@ -369,7 +349,7 @@ static int fc_config_add_dir(oconfig_item_t *ci) { else if (strcasecmp("Instance", option->key) == 0) status = fc_config_add_dir_instance(dir, option); else if (strcasecmp("Name", option->key) == 0) - status = fc_config_add_dir_name(dir, option); + status = cf_util_get_string(option, &dir->name); else if (strcasecmp("MTime", option->key) == 0) status = fc_config_add_dir_mtime(dir, option); else if (strcasecmp("Size", option->key) == 0)