stats plugin: Don't check the value of '(value_t).derive'.
authorFlorian Forster <octo@collectd.org>
Fri, 26 Jul 2013 13:18:43 +0000 (15:18 +0200)
committerFlorian Forster <octo@collectd.org>
Fri, 26 Jul 2013 13:18:43 +0000 (15:18 +0200)
statsd_parse_value() will write to (value_t).gauge, so don't read from another
member of the union. This also fixes problems with counter values being zero,
for whatever that is supposed to be useful for.

Github: #358

src/statsd.c

index b697a34..5af3483 100644 (file)
@@ -218,9 +218,6 @@ static int statsd_handle_counter (char const *name, /* {{{ */
   if (status != 0)
     return (status);
 
-  if (value.derive < 1)
-    return (-1);
-
   return (statsd_metric_add (name, (double) (value.gauge / scale.gauge),
         STATSD_COUNTER));
 } /* }}} int statsd_handle_counter */