diff options
| author | Samuel Dobson <[email protected]> | 2020-09-13 11:50:18 +1200 |
|---|---|---|
| committer | Samuel Dobson <[email protected]> | 2020-09-13 12:04:43 +1200 |
| commit | be375b2206fef8a5b6860eed14daab5f82386da4 (patch) | |
| tree | d47f89cc89541a9dbfddc9ea863a5d3d8babc794 /src/script/script.cpp | |
| parent | Merge #19922: test: Run rpc_txoutproof.py even with wallet disabled (diff) | |
| parent | bugfix: make LoadWallet assigns status always (diff) | |
| download | discoin-be375b2206fef8a5b6860eed14daab5f82386da4.tar.xz discoin-be375b2206fef8a5b6860eed14daab5f82386da4.zip | |
Merge #19919: bugfix: make LoadWallet assigns status always
8b39a875581bed1c2f40a7d9616bdb7cc642bf59 bugfix: make LoadWallet assigns status always (Akio Nakamura)
Pull request description:
In my enviroment, ```test/functional/wallet_multiwallet.py``` failed in line 237 for master( 147d50d63 ).
It got an expected rpc-error-message, but error code was not (-4) but (-18).
This is because that although loadwallet() in rpcwallet.cpp assumes LoadWallet() always assign some value to the 'status', but LoadWallet() does not do so in some situation.
This PR intends to fix above and prevends loadwallet() returns ambiguous error code.
ACKs for top commit:
hebasto:
re-ACK 8b39a875581bed1c2f40a7d9616bdb7cc642bf59, that is the same as 1728059730abef04f3fa84de0b6e20044be7a9d6.
ryanofsky:
Code review ACK 8b39a875581bed1c2f40a7d9616bdb7cc642bf59 (same as previous)
meshcollider:
utACK 8b39a875581bed1c2f40a7d9616bdb7cc642bf59
Tree-SHA512: a75d8240f60325bfdb69a07d392269fec97de743f38fe108371eb63a0aba5d8ce3cc484ecc69e81febf8040f5ab64f3a9450b98f8e07a0c17803784bb6f342bf
Diffstat (limited to 'src/script/script.cpp')
0 files changed, 0 insertions, 0 deletions