diff options
| author | Wladimir J. van der Laan <laanwj@gmail.com> | 2017-09-26 16:16:46 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2017-09-26 16:16:50 +0200 |
| commit | 0f8e09599db6fd34fa5d83160620cb2ddb05f09f (patch) | |
| tree | 58ec91131f64a18d44ff1ec059c3a8675a443273 /src/wallet/wallet.cpp | |
| parent | 67879b7c443144e479c763f64abb9c1ed7e44eaa (diff) | |
| parent | df10edfd03fe4c9d1eca43a17ac4c54a14ac4832 (diff) | |
Merge #11288: More user-friendly error message when partially signing
df10edf More user-friendly error message when partially signing (MeshCollider)
Pull request description:
When partially signing a transaction using `signrawtransaction`, if the wallet doesn't have access to a key, it will output a scary error message `"error": "Operation not valid with the current stack size"`, yet it will partially sign the transaction anyway. This puts a lot of users off, because they don't realise the signing actually succeeded for some inputs. This catches that specific error when signing, and outputs a friendlier message which says `Unable to sign input, invalid stack size (possibly missing key)`.
This is the best way I could think of to fix the issue, but please let me know if you come up with a better way to do it :)
Fixes https://github.com/bitcoin/bitcoin/issues/9988
Tree-SHA512: 65e1d4a49caa4202e1357b0b3f42329d76456c7b4286d63232226e03267809027b0c44e0faaa1da8b86c9ad677e3a3d655698a24fc870d6a661203c9f56ef95b
Diffstat (limited to 'src/wallet/wallet.cpp')
0 files changed, 0 insertions, 0 deletions