From: Nick Porter <nick@portercomputing.co.uk>
Date: Tue, 30 May 2023 11:41:43 +0000 (+0100)
Subject: Correct maths in redis_ippool_tool statistics
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=72b28c1e8649da35a7af5aa1e8e764f1162da691;p=thirdparty%2Ffreeradius-server.git

Correct maths in redis_ippool_tool statistics
---

diff --git a/src/modules/rlm_redis_ippool/rlm_redis_ippool_tool.c b/src/modules/rlm_redis_ippool/rlm_redis_ippool_tool.c
index 12b9ea48d85..b6973c10bb2 100644
--- a/src/modules/rlm_redis_ippool/rlm_redis_ippool_tool.c
+++ b/src/modules/rlm_redis_ippool/rlm_redis_ippool_tool.c
@@ -1414,8 +1414,6 @@ do { \
 		}
 
 		for (i = 0; i < (size_t)slen; i++) {
-			uint64_t acum = 0;
-
 			if (driver_get_stats(&stats, conf->driver,
 					     pools[i], talloc_array_length(pools[i])) < 0) fr_exit_now(EXIT_FAILURE);
 
@@ -1431,12 +1429,9 @@ do { \
 				INFO("used (%%)         : 0");
 			}
 			INFO("expiring 0-1m    : %" PRIu64, stats.expiring_1m);
-			acum += stats.expiring_1m;
-			INFO("expiring 1-30m   : %" PRIu64, stats.expiring_30m - acum);
-			acum += stats.expiring_30m;
-			INFO("expiring 30m-1h  : %" PRIu64, stats.expiring_1h - acum);
-			acum += stats.expiring_1h;
-			INFO("expiring 1h-1d   : %" PRIu64, stats.expiring_1d - acum);
+			INFO("expiring 1-30m   : %" PRIu64, stats.expiring_30m - stats.expiring_1m);
+			INFO("expiring 30m-1h  : %" PRIu64, stats.expiring_1h - stats.expiring_30m);
+			INFO("expiring 1h-1d   : %" PRIu64, stats.expiring_1d - stats.expiring_1h);
 			INFO("--");
 		}
 	}