HomePhabricator

Disallow using addresses in createmultisig

Description

Disallow using addresses in createmultisig

Summary:
Make createmultisig only accept public keys with the old functionality
marked as deprecated.

Splits _createmultisig_redeemscript into two functions, one for
getting public keys from UniValue and one for getting addresses
from UniValue and then their respective public keys. The one for
retrieving address's public keys is located in rpcwallet.cpp

Adds a test to deprecated_rpc.py for testing the deprecation.

Update the tests to give only public keys to createmultisig.

Depends on D1967 and D2001

Backport of Core PR11415

Test Plan:

make VERBOSE=1 check && ./test/functional/test_runner.py

Reviewers: #bitcoin_abc, schancel, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien, teamcity

Differential Revision: https://reviews.bitcoinabc.org/D2002

Details

Provenance
Andrew Chow <achow101-github@achow101.com>Authored on Jan 3 2019, 22:56
deadalnixCommitted on May 6 2019, 22:34
deadalnixPushed on May 6 2019, 22:34
Reviewer
Restricted Project
Differential Revision
D2002: Disallow using addresses in createmultisig
Parents
rABCf5919276fb6e: Merge #10027: Set to nullptr after delete
Branches
Unknown
Tags
Unknown

Event Timeline