Code

exec plugin: Print a warning when the returned string is incorrect.
authorFlorian Forster <octo@leeloo.lan.home.verplant.org>
Fri, 25 May 2007 17:48:56 +0000 (19:48 +0200)
committerFlorian Forster <octo@leeloo.lan.home.verplant.org>
Fri, 25 May 2007 17:48:56 +0000 (19:48 +0200)
Also the debug messages have been beautified.

src/exec.c

index 518efc870739169381d1ab51848bc3ec23de42f2..bd96e07b0d2b700efc92ed6b90b77b04ff576a81 100644 (file)
@@ -273,9 +273,15 @@ static void *exec_read_one (void *arg)
     char *type_instance;
     char *value;
 
-    DEBUG ("buffer = %s", buffer);
-
     len = strlen (buffer);
+
+    /* Remove newline from end. */
+    while ((len > 0) && ((buffer[len - 1] == '\n')
+         || (buffer[len - 1] == '\r')))
+      buffer[--len] = '\0';
+
+    DEBUG ("exec plugin: exec_read_one: buffer = %s", buffer);
+
     if (len < 5)
       continue;
 
@@ -299,11 +305,15 @@ static void *exec_read_one (void *arg)
 
     value = strchr (type_instance, ',');
     if (value == NULL)
+    {
+      WARNING ("exec plugin: type-instance is missing.");
       continue;
+    }
     *value = '\0';
     value++;
 
-    DEBUG ("value = %s", value);
+    DEBUG ("exec plugin: exec_read_one: type = %s; type_instance = %s; "
+       "value = %s;", type, type_instance, value);
 
     if (strcasecmp ("counter", type) == 0)
       submit_counter (type_instance, atoll (value));