X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Futils_format_json.c;h=898b1725eac821acc99407c09061661abbeabca7;hb=eb342c120baa5d015d71ae8ec8fba490df88ee64;hp=ec198438abb8f064b8cedfadefaac4841fd6aba6;hpb=f5dd22c7af44798d3512d62f733b4e44a6c4ddb0;p=collectd.git diff --git a/src/utils_format_json.c b/src/utils_format_json.c index ec198438..898b1725 100644 --- a/src/utils_format_json.c +++ b/src/utils_format_json.c @@ -152,7 +152,7 @@ static int values_to_json (char *buffer, size_t buffer_size, /* {{{ */ } /* }}} int values_to_json */ static int dstypes_to_json (char *buffer, size_t buffer_size, /* {{{ */ - const data_set_t *ds, const value_list_t *vl) + const data_set_t *ds) { size_t offset = 0; int i; @@ -189,7 +189,7 @@ static int dstypes_to_json (char *buffer, size_t buffer_size, /* {{{ */ } /* }}} int dstypes_to_json */ static int dsnames_to_json (char *buffer, size_t buffer_size, /* {{{ */ - const data_set_t *ds, const value_list_t *vl) + const data_set_t *ds) { size_t offset = 0; int i; @@ -234,7 +234,10 @@ static int meta_data_to_json (char *buffer, size_t buffer_size, /* {{{ */ int status; int i; - memset (buffer, 0, buffer_size); + buffer[0] = 0; + + if (meta == NULL) + return (EINVAL); #define BUFFER_ADD(...) do { \ status = ssnprintf (buffer + offset, buffer_size - offset, \ @@ -248,6 +251,12 @@ static int meta_data_to_json (char *buffer, size_t buffer_size, /* {{{ */ } while (0) keys_num = meta_data_toc (meta, &keys); + if (keys_num == 0) + { + sfree (keys); + return (0); + } + for (i = 0; i < keys_num; ++i) { int type; @@ -303,7 +312,7 @@ static int meta_data_to_json (char *buffer, size_t buffer_size, /* {{{ */ #undef BUFFER_ADD return (0); -} /* int meta_data_to_json */ +} /* }}} int meta_data_to_json */ static int value_list_to_json (char *buffer, size_t buffer_size, /* {{{ */ const data_set_t *ds, const value_list_t *vl, int store_rates) @@ -334,12 +343,12 @@ static int value_list_to_json (char *buffer, size_t buffer_size, /* {{{ */ return (status); BUFFER_ADD ("\"values\":%s", temp); - status = dstypes_to_json (temp, sizeof (temp), ds, vl); + status = dstypes_to_json (temp, sizeof (temp), ds); if (status != 0) return (status); BUFFER_ADD (",\"dstypes\":%s", temp); - status = dsnames_to_json (temp, sizeof (temp), ds, vl); + status = dsnames_to_json (temp, sizeof (temp), ds); if (status != 0) return (status); BUFFER_ADD (",\"dsnames\":%s", temp);