[collectd] [PATCH] Fix conceptual buglet in default_callback

David Gibson david at gibson.dropbear.id.au
Wed Aug 10 12:07:14 CEST 2016


The 'DERIVE' path in default_callback() increments the 'counter' field of
value instead of the 'derive' field.  Since those fields have the same
type and offset within the value union this will still work fine, but
AFAICT it's conceptually incorrect.  This corrects it.

Signed-off-by: David Gibson <david at gibson.dropbear.id.au>
---
 src/daemon/utils_match.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Came across this while looking at some other things.

diff --git a/src/daemon/utils_match.c b/src/daemon/utils_match.c
index 5273c90..914b6e2 100644
--- a/src/daemon/utils_match.c
+++ b/src/daemon/utils_match.c
@@ -170,7 +170,7 @@ static int default_callback (const char __attribute__((unused)) *str,
 
     if (data->ds_type & UTILS_MATCH_CF_DERIVE_INC)
     {
-      data->value.counter++;
+      data->value.derive++;
       data->values_num++;
       return (0);
     }
-- 
2.7.4




More information about the collectd mailing list