Code

Bugfix for libcollectdclient lcc_putval.
authorJohan Van den Brande <johan@vandenbrande.com>
Wed, 19 May 2010 10:23:23 +0000 (12:23 +0200)
committerFlorian Forster <octo@huhu.verplant.org>
Wed, 19 May 2010 10:25:04 +0000 (12:25 +0200)
Hello,

I'm using collectd to monitor system and process parameters of our code under
test running on a cluster. The test processes communicate to collectd via the
unixsock plugin, but there was a small error in the client library which I had
to fix first.

The wire format of the putval command is missing a space behind the closing
double quote of the identifier. As I said, it is a small fix, but nevertheless
I attached the patch for client.c. The bug is present in 4.6 (that is the
version I'm using in production), but I fixed it against 4.10.0.

Cheers,
Johan

Signed-off-by: Florian Forster <octo@huhu.verplant.org>
src/libcollectdclient/client.c

index 63d4e9d7cd75bf9d6a7652d7241559183fa98e39..0c748ba7fa47e3b30f1d2a8d5c30dffdfd70d3e3 100644 (file)
@@ -777,9 +777,9 @@ int lcc_putval (lcc_connection_t *c, const lcc_value_list_t *vl) /* {{{ */
     SSTRCATF (command, " interval=%i", vl->interval);
 
   if (vl->time > 0)
-    SSTRCATF (command, "%u", (unsigned int) vl->time);
+    SSTRCATF (command, " %u", (unsigned int) vl->time);
   else
-    SSTRCAT (command, "N");
+    SSTRCAT (command, " N");
 
   for (i = 0; i < vl->values_len; i++)
   {