saturating math

coriolinus · shawntabrizi · web-flow · commit c503fa763b38 · 2021-05-31T10:38:54.000+02:00
Co-authored-by: Shawn Tabrizi &lt;shawntabrizi@gmail.com&gt;
diff --git a/frame/election-provider-multi-phase/src/signed.rs b/frame/election-provider-multi-phase/src/signed.rs
@@ -249,10 +249,10 @@ impl<T: Config> Pallet<T> {
 		let encoded_len: BalanceOf<T> = encoded_len.into();
 		let feasibility_weight = Self::feasibility_weight_of(solution, size);
 
-		let len_deposit = T::SignedDepositByte::get() * encoded_len;
-		let weight_deposit = T::SignedDepositWeight::get() * feasibility_weight.saturated_into();
+		let len_deposit = T::SignedDepositByte::get().saturating_mul(encoded_len);
+		let weight_deposit = T::SignedDepositWeight::get().saturating_mul(feasibility_weight.saturated_into());
 
-		T::SignedDepositBase::get() + len_deposit + weight_deposit
+		T::SignedDepositBase::get().saturating_add(len_deposit).saturating_add(weight_deposit)
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -249,10 +249,10 @@ impl<T: Config> Pallet<T> {`
`249`	`249`	`let encoded_len: BalanceOf<T> = encoded_len.into();`
`250`	`250`	`let feasibility_weight = Self::feasibility_weight_of(solution, size);`
`251`	`251`
`252`		`- let len_deposit = T::SignedDepositByte::get() * encoded_len;`
`253`		`- let weight_deposit = T::SignedDepositWeight::get() * feasibility_weight.saturated_into();`
	`252`	`+ let len_deposit = T::SignedDepositByte::get().saturating_mul(encoded_len);`
	`253`	`+ let weight_deposit = T::SignedDepositWeight::get().saturating_mul(feasibility_weight.saturated_into());`
`254`	`254`
`255`		`- T::SignedDepositBase::get() + len_deposit + weight_deposit`
	`255`	`+ T::SignedDepositBase::get().saturating_add(len_deposit).saturating_add(weight_deposit)`
`256`	`256`	`}`
`257`	`257`	`}`
`258`	`258`