From bf0776e3dd8f457ec5c4b368d05ffb09d2cda20d Mon Sep 17 00:00:00 2001 From: "Tobin C. Harding" Date: Mon, 26 May 2025 07:37:51 +0100 Subject: [PATCH] Remove panic using checked arithmetic During dev I introduced a pancic, remove it. --- units/src/fee.rs | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/units/src/fee.rs b/units/src/fee.rs index f8a693351..63a75cbfb 100644 --- a/units/src/fee.rs +++ b/units/src/fee.rs @@ -31,10 +31,14 @@ impl Amount { return None; } - let sats = self.to_sat() * 1_000; // Because we use per/kwu. - let fee_rate = sats / wu; - - FeeRate::from_sat_per_kwu(fee_rate) + // Mul by 1,000 because we use per/kwu. + match self.to_sat().checked_mul(1_000) { + Some(sats) => { + let fee_rate = sats / wu; + FeeRate::from_sat_per_kwu(fee_rate) + } + None => None, + } } /// Checked weight ceiling division.