Olaoluwa Osuntokun af4afdb6f0 lnwallet: use btcutil.NewAmount for BTC -> SAT conversions ...

In this commit, we fix an existing rounding related bug in the codebase.
The RPC interface for btcd and bitcoind return values in BTC rather than
in satoshis. So in several places, we're forced to convert ourselves
manually. The existing logic attempted to do this, but didn't properly
account for rounding. As a result, our values can be off due to not
rounding incorrectly.

The fix for this is easy: simply properly use btcutil.NewAmount
everywhere which does rounding properly.

Fixes #939.

2018-03-25 19:15:18 -07:00

68 KiB

Raw Blame History

View Raw