diff options
author | MarcoFalke <falke.marco@gmail.com> | 2018-03-23 19:03:31 -0400 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2018-03-23 19:03:34 -0400 |
commit | 02b7e8319aef2a870264ad4fa2e3bb18664dcc36 (patch) | |
tree | b1d7db94942824695c7e136f4b4719e8724ef924 /src/validation.cpp | |
parent | cead84b72d27517338ac3c49eaaed4db07352f41 (diff) | |
parent | 6acb02d635da68c0506b49a8dd27c2c3ded4addf (diff) | |
download | bitcoin-02b7e8319aef2a870264ad4fa2e3bb18664dcc36.tar.xz |
Merge #12709: [wallet] shuffle sendmany recipients ordering
6acb02d635 add release note for sendmany output shuffling (Gregory Sanders)
cf6ef3c139 shuffle sendmany recipients ordering to shuffle tx outputs (Gregory Sanders)
Pull request description:
Unless there is something important I'm missing, we're just possible leaking information by preserving whatever ordering json object ordering is giving us (no guarantees at all).
This is unneeded for `sendtoaddress` since there is only 1 or 2 outputs, and the change output is shuffled in.
This will not effect `*raw` behavior by design, since users generally want full control using those apis. Further PRs could add optional args to over-ride that behavior.
Alternative ideas would be to sort the outputs by some deterministic ordering. (this would require more refactoring since change outputs are created and handled by caller)
related: https://github.com/bitcoin/bitcoin/pull/12699
Tree-SHA512: afdd990dde6a4a9e7eef7bb2e3342a46d11900d7fe6e6e4eb0cc6b5deed89df989fa7931a4bdcbf49b7c2d7a13c90169af3a166466e5760948bacabe3490f572
Diffstat (limited to 'src/validation.cpp')
0 files changed, 0 insertions, 0 deletions