gaspersic/solana
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Prevent u64 overflow when calculating current stake percentage (#15453)

Browse Source 
(cherry picked from commit 5ae37b9675)

Co-authored-by: Michael Vines <mvines@gmail.com>
This commit is contained in:
mergify[bot]
2021-02-20 08:11:45 +00:00
committed by GitHub
parent de4d2e640e
commit ddef2fb7fa
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
watchtower/src/main.rs
View File

@ -241,9 +241,9 @@ fn main() -> Result<(), Box<dyn error::Error>> {
.sum(); .sum();
let total_stake = total_current_stake + total_delinquent_stake; let total_stake = total_current_stake + total_delinquent_stake;
let current_stake_percent = total_current_stake * 100 / total_stake; let current_stake_percent = total_current_stake as f64 * 100. / total_stake as f64;
info!( info!(
"Current stake: {}% | Total stake: {}, current stake: {}, delinquent: {}", "Current stake: {:.2}% | Total stake: {}, current stake: {}, delinquent: {}",
current_stake_percent, current_stake_percent,
Sol(total_stake), Sol(total_stake),
Sol(total_current_stake), Sol(total_current_stake),
@ -271,10 +271,10 @@ fn main() -> Result<(), Box<dyn error::Error>> {
)); ));
} }
if config.monitor_active_stake && current_stake_percent < 80 { if config.monitor_active_stake && current_stake_percent < 80. {
failures.push(( failures.push((
"current-stake", "current-stake",
format!("Current stake is {}%", current_stake_percent), format!("Current stake is {:.2}%", current_stake_percent),
)); ));
} }