diff options
author | Stelios Fragkakis <52996999+stelfrag@users.noreply.github.com> | 2023-12-14 10:50:48 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-12-14 10:50:48 +0200 |
commit | e3b4ddedd2ffd557a5277fe0a3db78f025b22e2a (patch) | |
tree | c8a04a4a86b45d89a21439adfc9db73e42a11b6b /database | |
parent | 9599f17b89c6e3a3c8a906f27751eda210ddf75c (diff) |
Fix coverity issues (#16596)
* Fix coverity issues
* Prevent potential overflow
Diffstat (limited to 'database')
-rwxr-xr-x | database/engine/rrdengineapi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/database/engine/rrdengineapi.c b/database/engine/rrdengineapi.c index 853fd0e6ea..ae9ee24cc1 100755 --- a/database/engine/rrdengineapi.c +++ b/database/engine/rrdengineapi.c @@ -811,13 +811,13 @@ static bool rrdeng_load_page_next(struct storage_engine_query_handle *rrddim_han } else { position = (handle->now_s - page_start_time_s) * (entries - 1) / (page_end_time_s - page_start_time_s); - time_t point_end_time_s = page_start_time_s + position * page_update_every_s; + time_t point_end_time_s = page_start_time_s + position * (time_t) page_update_every_s; while(point_end_time_s < handle->now_s && position + 1 < entries) { // https://github.com/netdata/netdata/issues/14411 // we really need a while() here, because the delta may be // 2 points at higher tiers position++; - point_end_time_s = page_start_time_s + position * page_update_every_s; + point_end_time_s = page_start_time_s + position * (time_t) page_update_every_s; } handle->now_s = point_end_time_s; } |