projects
/
collectd.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
(from parent 1:
d89088e
)
Merge pull request #655 from dothebart/statsd_fix_division_by_zero
author
Pierre-Yves Ritschard
<pyr@spootnik.org>
Tue, 29 Jul 2014 12:04:41 +0000
(14:04 +0200)
committer
Pierre-Yves Ritschard
<pyr@spootnik.org>
Tue, 29 Jul 2014 12:04:41 +0000
(14:04 +0200)
Statsd: avoid doing a division by zero when calculating the average
src/utils_latency.c
patch
|
blob
|
history
diff --git
a/src/utils_latency.c
b/src/utils_latency.c
index
94da211
..
7699f57
100644
(file)
--- a/
src/utils_latency.c
+++ b/
src/utils_latency.c
@@
-128,7
+128,7
@@
cdtime_t latency_counter_get_average (latency_counter_t *lc) /* {{{ */
{
double average;
- if (
lc == NULL
)
+ if (
(lc == NULL) || (lc->num == 0)
)
return (0);
average = CDTIME_T_TO_DOUBLE (lc->sum) / ((double) lc->num);
@@
-146,7
+146,7
@@
cdtime_t latency_counter_get_percentile (latency_counter_t *lc,
int sum;
size_t i;
- if ((lc == NULL) || !((percent > 0.0) && (percent < 100.0)))
+ if ((lc == NULL) ||
(lc->num == 0) ||
!((percent > 0.0) && (percent < 100.0)))
return (0);
/* Find index i so that at least "percent" events are within i+1 ms. */