[collectd] [PATCH] Fix conceptual buglet in default_callback

[David Gibson]

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