diff options
authorJacob Erlbeck <jerlbeck@sysmocom.de>2015-11-09 11:39:42 +0100
committerJacob Erlbeck <jerlbeck@sysmocom.de>2015-11-09 14:49:52 +0100
commit8a97cb911122f112a1a829f6f5d2be3922153a74 (patch)
parentaed7c125fdfe743a78cb3fd8123af5257a900df9 (diff)
stats: Fix handling of multiple resporters
Currently the reporter loop is aborted, if the event would be ignored due to an insufficient max_level. Thus the reporters that happen to be stored afterwards would be skipped even if their level were sufficient. This commit replaces the conditional 'return' statement by a considtional 'continue' statement to ensure the processing of the list tail. Sponsored-by: On-Waves ehf
1 files changed, 2 insertions, 2 deletions
diff --git a/src/stats.c b/src/stats.c
index 0a1a1486..d83fd87d 100644
--- a/src/stats.c
+++ b/src/stats.c
@@ -593,7 +593,7 @@ static int rate_ctr_handler(
if (!osmo_stats_reporter_check_config(srep,
ctrg->idx, ctrg->desc->class_id))
- return 0;
+ continue;
osmo_stats_reporter_send_counter(srep, ctrg, desc,
ctr->current, delta);
@@ -647,7 +647,7 @@ static int osmo_stat_item_handler(
if (!osmo_stats_reporter_check_config(srep,
statg->idx, statg->desc->class_id))
- return 0;
+ continue;
osmo_stats_reporter_send_item(srep, statg,
item->desc, value);