Replace ismine with producesignature check in witnessifier
Instead of using ismine to check whether an address can be spent by us, make the witness version of the script or address first and then use ProduceSignature with the DummySignatureCreator to check if we can solve for the script. Also fixes test cases to reflect this change.
This commit is contained in:
parent
9edda0c5f5
commit
e222dc2aee
2 changed files with 22 additions and 13 deletions
|
@ -1067,11 +1067,15 @@ public:
|
|||
bool operator()(const CKeyID &keyID) {
|
||||
if (pwallet) {
|
||||
CScript basescript = GetScriptForDestination(keyID);
|
||||
isminetype typ;
|
||||
typ = IsMine(*pwallet, basescript, SIGVERSION_WITNESS_V0);
|
||||
if (typ != ISMINE_SPENDABLE && typ != ISMINE_WATCH_SOLVABLE)
|
||||
return false;
|
||||
CScript witscript = GetScriptForWitness(basescript);
|
||||
SignatureData sigs;
|
||||
// This check is to make sure that the script we created can actually be solved for and signed by us
|
||||
// if we were to have the private keys. This is just to make sure that the script is valid and that,
|
||||
// if found in a transaction, we would still accept and relay that transcation.
|
||||
if (!ProduceSignature(DummySignatureCreator(pwallet), witscript, sigs) ||
|
||||
!VerifyScript(sigs.scriptSig, witscript, &sigs.scriptWitness, MANDATORY_SCRIPT_VERIFY_FLAGS | SCRIPT_VERIFY_WITNESS_PUBKEYTYPE, DummySignatureCreator(pwallet).Checker())) {
|
||||
return false;
|
||||
}
|
||||
pwallet->AddCScript(witscript);
|
||||
result = CScriptID(witscript);
|
||||
return true;
|
||||
|
@ -1088,11 +1092,15 @@ public:
|
|||
result = scriptID;
|
||||
return true;
|
||||
}
|
||||
isminetype typ;
|
||||
typ = IsMine(*pwallet, subscript, SIGVERSION_WITNESS_V0);
|
||||
if (typ != ISMINE_SPENDABLE && typ != ISMINE_WATCH_SOLVABLE)
|
||||
return false;
|
||||
CScript witscript = GetScriptForWitness(subscript);
|
||||
SignatureData sigs;
|
||||
// This check is to make sure that the script we created can actually be solved for and signed by us
|
||||
// if we were to have the private keys. This is just to make sure that the script is valid and that,
|
||||
// if found in a transaction, we would still accept and relay that transcation.
|
||||
if (!ProduceSignature(DummySignatureCreator(pwallet), witscript, sigs) ||
|
||||
!VerifyScript(sigs.scriptSig, witscript, &sigs.scriptWitness, MANDATORY_SCRIPT_VERIFY_FLAGS | SCRIPT_VERIFY_WITNESS_PUBKEYTYPE, DummySignatureCreator(pwallet).Checker())) {
|
||||
return false;
|
||||
}
|
||||
pwallet->AddCScript(witscript);
|
||||
result = CScriptID(witscript);
|
||||
return true;
|
||||
|
|
|
@ -459,13 +459,14 @@ class SegWitTest(BitcoinTestFramework):
|
|||
self.mine_and_test_listunspent(unsolvable_after_importaddress, 1)
|
||||
self.mine_and_test_listunspent(unseen_anytime, 0)
|
||||
|
||||
# addwitnessaddress should refuse to return a witness address if an uncompressed key is used or the address is
|
||||
# not in the wallet
|
||||
# addwitnessaddress should refuse to return a witness address if an uncompressed key is used
|
||||
# note that no witness address should be returned by unsolvable addresses
|
||||
# the multisig_without_privkey_address will fail because its keys were not added with importpubkey
|
||||
for i in uncompressed_spendable_address + uncompressed_solvable_address + unknown_address + unsolvable_address + [multisig_without_privkey_address]:
|
||||
for i in uncompressed_spendable_address + uncompressed_solvable_address + unknown_address + unsolvable_address:
|
||||
assert_raises_jsonrpc(-4, "Public key or redeemscript not known to wallet, or the key is uncompressed", self.nodes[0].addwitnessaddress, i)
|
||||
|
||||
# addwitnessaddress should return a witness addresses even if keys are not in the wallet
|
||||
self.nodes[0].addwitnessaddress(multisig_without_privkey_address)
|
||||
|
||||
for i in compressed_spendable_address + compressed_solvable_address:
|
||||
witaddress = self.nodes[0].addwitnessaddress(i)
|
||||
# addwitnessaddress should return the same address if it is a known P2SH-witness address
|
||||
|
@ -542,7 +543,7 @@ class SegWitTest(BitcoinTestFramework):
|
|||
# addwitnessaddress should refuse to return a witness address if an uncompressed key is used
|
||||
# note that a multisig address returned by addmultisigaddress is not solvable until it is added with importaddress
|
||||
# premature_witaddress are not accepted until the script is added with addwitnessaddress first
|
||||
for i in uncompressed_spendable_address + uncompressed_solvable_address + premature_witaddress + [compressed_solvable_address[1]]:
|
||||
for i in uncompressed_spendable_address + uncompressed_solvable_address + premature_witaddress:
|
||||
# This will raise an exception
|
||||
assert_raises_jsonrpc(-4, "Public key or redeemscript not known to wallet, or the key is uncompressed", self.nodes[0].addwitnessaddress, i)
|
||||
|
||||
|
|
Loading…
Reference in a new issue