X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Futils_format_graphite.c;h=e23d5d92c65599fecc29ac575ca48c68af2b3369;hb=d3db12b73a1c3c17a2dfc0ba96ec99490088c957;hp=83512015898e22b27c527b431aa03bdcda5f2865;hpb=e9fb3dcaad4d46f1e594fd773f6d2327822bdddf;p=collectd.git diff --git a/src/utils_format_graphite.c b/src/utils_format_graphite.c index 83512015..e23d5d92 100644 --- a/src/utils_format_graphite.c +++ b/src/utils_format_graphite.c @@ -29,6 +29,8 @@ #include "utils_cache.h" #include "utils_parse_option.h" +#define GRAPHITE_FORBIDDEN " \t\"\\:!/()\n\r" + /* Utils functions to format data sets in graphite format. * Largely taken from write_graphite.c as it remains the same formatting */ @@ -59,7 +61,7 @@ static int gr_format_values (char *ret, size_t ret_len, } while (0) if (ds->ds[ds_num].type == DS_TYPE_GAUGE) - BUFFER_ADD ("%f", vl->values[ds_num].gauge); + BUFFER_ADD (GAUGE_FORMAT, vl->values[ds_num].gauge); else if (rates != NULL) BUFFER_ADD ("%f", rates[ds_num]); else if (ds->ds[ds_num].type == DS_TYPE_COUNTER) @@ -169,6 +171,18 @@ static int gr_format_name (char *ret, int ret_len, return (0); } +static void escape_graphite_string (char *buffer, char escape_char) +{ + char *head; + + assert (strchr(GRAPHITE_FORBIDDEN, escape_char) == NULL); + + for (head = buffer + strcspn(buffer, GRAPHITE_FORBIDDEN); + *head != '\0'; + head += strcspn(head, GRAPHITE_FORBIDDEN)) + *head = escape_char; +} + int format_graphite (char *buffer, size_t buffer_size, data_set_t const *ds, value_list_t const *vl, char const *prefix, char const *postfix, char const escape_char, @@ -204,7 +218,7 @@ int format_graphite (char *buffer, size_t buffer_size, return (status); } - escape_string (key, sizeof (key)); + escape_graphite_string (key, escape_char); /* Convert the values to an ASCII representation and put that into * `values'. */ status = gr_format_values (values, sizeof (values), i, ds, vl, rates);