subtractfeefromamount=true fails with insufficient funds #25129

issue MarcoFalke opened this issue on May 13, 2022

MarcoFalke commented at 2:54 PM on May 13, 2022: member

Steps to reproduce:

wget https://github.com/bitcoin/bitcoin/files/8688722/datadir_node0.tar.gz
tar -xvf datadir_node0.tar.gz
export DD='-datadir=./node0/'
bitcoin-qt $DD &
bitcoin-cli $DD loadwallet rpc_online
bitcoin-cli $DD -named sendtoaddress address=bcrt1qqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqq3xueyj amount=1966.66779818 subtractfeefromamount=true

MarcoFalke added the label Bug on May 13, 2022
MarcoFalke added the label Wallet on May 13, 2022
MarcoFalke commented at 1:23 PM on May 16, 2022: member

Maybe this should be replaced by sendall in the tests, but I am wondering if the bug should be fixed or ignored.

cc @Xekyo @achow101 https://github.com/bitcoin/bitcoin/pull/24118
achow101 commented at 2:16 PM on May 16, 2022: member

The bug would probably be fixed by #23534, however I think changing to use sendall is the more correct fix.
Xekyo commented at 6:04 PM on May 16, 2022: member

Looking at how the test works, and seeing that it creates transactions sending random recipient amounts up to the full balance, it seems likely that it occasionally creates some tiny UTXOs that are skipped during AvailableCoins(…) due to having a negative effective value. Since line 448 where this error occurs is trying to empty the wallet, using sendall seems like the appropriate fix. IIRC, we also allow creating a PSBT with sendall, so it would work for both 444 and 448. Might be an "Easy first issue"?
MarcoFalke added the label good first issue on May 17, 2022
MarcoFalke commented at 6:38 AM on May 17, 2022: member
yeah, changing the test is a gfi

Useful skills:
- Bitcoin Core functional test framework and RPC methods
Want to work on this issue?

For guidance on contributing, please read CONTRIBUTING.md before opening your pull request.
ishaanam referenced this in commit 5bb384638e on May 17, 2022
ishaanam referenced this in commit c6122f560b on May 17, 2022
achow101 closed this on May 17, 2022
achow101 referenced this in commit f7a1e676d5 on May 17, 2022
naumenkogs referenced this in commit 59f6c138bc on May 23, 2022
sidhujag referenced this in commit 40c71c94cd on May 28, 2022
ekkis commented at 5:24 PM on June 30, 2022: none

I have this issue in trying to use bitcoin-cli but there is no sendall command. suggestions?
achow101 commented at 6:00 PM on June 30, 2022: member

sendall will be available in the next major release. You can build the master branch from source if you need it now.

Otherwise you can achieve the same thing by manually choosing all of the utxos in your wallet. listunspent will provide you the utxos and you can use them with send or the raw transaction or psbt workflows.
ekkis commented at 6:03 PM on June 30, 2022: none

wonderful. thank you
DrahtBot locked this on Jun 30, 2023

Contributors

Labels

Bug Wallet good first issue

Linked (view graph)

#25030 wallet_taproot.py fails with "Insufficient funds (-4)"#25159 test: use `sendall` in wallet_taproot.py tests

subtractfeefromamount=true fails with insufficient funds #25129

Useful skills:

Want to work on this issue?