diff options
author | Sjors Provoost <sjors@sprovoost.nl> | 2019-10-23 15:11:25 +0200 |
---|---|---|
committer | Sjors Provoost <sjors@sprovoost.nl> | 2020-02-04 11:20:25 +0100 |
commit | 709f8685ac37510aa145ac259753583c82280038 (patch) | |
tree | cd2ca5aa826166b1fc059d21402c0e6e0e7ebc0b | |
parent | 5efc25f9638866941028454cfa9bae27f1519cb4 (diff) |
[wallet] CreateTransaction: simplify change address check
-rw-r--r-- | src/wallet/wallet.cpp | 11 | ||||
-rwxr-xr-x | test/functional/rpc_fundrawtransaction.py | 2 |
2 files changed, 3 insertions, 10 deletions
diff --git a/src/wallet/wallet.cpp b/src/wallet/wallet.cpp index b53c251131..8e20f48ae3 100644 --- a/src/wallet/wallet.cpp +++ b/src/wallet/wallet.cpp @@ -2630,18 +2630,11 @@ bool CWallet::CreateTransaction(interfaces::Chain::Lock& locked_chain, const std // post-backup change. // Reserve a new key pair from key pool - if (!CanGetAddresses(true)) { - strFailReason = _("Can't generate a change-address key. No keys in the internal keypool and can't generate any keys.").translated; - return false; - } CTxDestination dest; - bool ret = reservedest.GetReservedDestination(dest, true); - if (!ret) - { - strFailReason = _("Keypool ran out, please call keypoolrefill first").translated; + if (!reservedest.GetReservedDestination(dest, true)) { + strFailReason = _("Can't generate a change-address key. Please call keypoolrefill first.").translated; return false; } - scriptChange = GetScriptForDestination(dest); } CTxOut change_prototype_txout(0, scriptChange); diff --git a/test/functional/rpc_fundrawtransaction.py b/test/functional/rpc_fundrawtransaction.py index 6f1ae0d3ba..8a2dc7e916 100755 --- a/test/functional/rpc_fundrawtransaction.py +++ b/test/functional/rpc_fundrawtransaction.py @@ -504,7 +504,7 @@ class RawTransactionsTest(BitcoinTestFramework): rawtx = self.nodes[1].createrawtransaction(inputs, outputs) # fund a transaction that requires a new key for the change output # creating the key must be impossible because the wallet is locked - assert_raises_rpc_error(-4, "Keypool ran out, please call keypoolrefill first", self.nodes[1].fundrawtransaction, rawtx) + assert_raises_rpc_error(-4, "Can't generate a change-address key. Please call keypoolrefill first.", self.nodes[1].fundrawtransaction, rawtx) # Refill the keypool. self.nodes[1].walletpassphrase("test", 100) |