Skip to content

Commit f4691ab

Browse files
committed
[RPC] Add wallet support for witness transactions (using P2SH)
Includes support for pushkeyhash wit v0 by Alex Morcos.
1 parent 605e847 commit f4691ab

File tree

2 files changed

+118
-0
lines changed

2 files changed

+118
-0
lines changed

src/rpc/misc.cpp

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -312,6 +312,43 @@ UniValue createmultisig(const UniValue& params, bool fHelp)
312312
return result;
313313
}
314314

315+
UniValue createwitnessaddress(const UniValue& params, bool fHelp)
316+
{
317+
if (fHelp || params.size() < 1 || params.size() > 1)
318+
{
319+
string msg = "createwitnessaddress \"script\"\n"
320+
"\nCreates a witness address for a particular script.\n"
321+
"It returns a json object with the address and witness script.\n"
322+
323+
"\nArguments:\n"
324+
"1. \"script\" (string, required) A hex encoded script\n"
325+
326+
"\nResult:\n"
327+
"{\n"
328+
" \"address\":\"multisigaddress\", (string) The value of the new address (P2SH of witness script).\n"
329+
" \"witnessScript\":\"script\" (string) The string value of the hex-encoded witness script.\n"
330+
"}\n"
331+
;
332+
throw runtime_error(msg);
333+
}
334+
335+
if (!IsHex(params[0].get_str())) {
336+
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Script must be hex-encoded");
337+
}
338+
339+
std::vector<unsigned char> code = ParseHex(params[0].get_str());
340+
CScript script(code.begin(), code.end());
341+
CScript witscript = GetScriptForWitness(script);
342+
CScriptID witscriptid(witscript);
343+
CBitcoinAddress address(witscriptid);
344+
345+
UniValue result(UniValue::VOBJ);
346+
result.push_back(Pair("address", address.ToString()));
347+
result.push_back(Pair("witnessScript", HexStr(witscript.begin(), witscript.end())));
348+
349+
return result;
350+
}
351+
315352
UniValue verifymessage(const UniValue& params, bool fHelp)
316353
{
317354
if (fHelp || params.size() != 3)
@@ -445,6 +482,7 @@ static const CRPCCommand commands[] =
445482
{ "control", "getinfo", &getinfo, true }, /* uses wallet if enabled */
446483
{ "util", "validateaddress", &validateaddress, true }, /* uses wallet if enabled */
447484
{ "util", "createmultisig", &createmultisig, true },
485+
{ "util", "createwitnessaddress", &createwitnessaddress, true },
448486
{ "util", "verifymessage", &verifymessage, true },
449487
{ "util", "signmessagewithprivkey", &signmessagewithprivkey, true },
450488

src/wallet/rpcwallet.cpp

Lines changed: 80 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1011,6 +1011,85 @@ UniValue addmultisigaddress(const UniValue& params, bool fHelp)
10111011
return CBitcoinAddress(innerID).ToString();
10121012
}
10131013

1014+
class Witnessifier : public boost::static_visitor<bool>
1015+
{
1016+
public:
1017+
CScriptID result;
1018+
1019+
bool operator()(const CNoDestination &dest) const { return false; }
1020+
1021+
bool operator()(const CKeyID &keyID) {
1022+
CPubKey pubkey;
1023+
if (pwalletMain && pwalletMain->GetPubKey(keyID, pubkey)) {
1024+
CScript basescript;
1025+
basescript << ToByteVector(pubkey) << OP_CHECKSIG;
1026+
CScript witscript = GetScriptForWitness(basescript);
1027+
pwalletMain->AddCScript(witscript);
1028+
result = CScriptID(witscript);
1029+
return true;
1030+
}
1031+
return false;
1032+
}
1033+
1034+
bool operator()(const CScriptID &scriptID) {
1035+
CScript subscript;
1036+
if (pwalletMain && pwalletMain->GetCScript(scriptID, subscript)) {
1037+
int witnessversion;
1038+
std::vector<unsigned char> witprog;
1039+
if (subscript.IsWitnessProgram(witnessversion, witprog)) {
1040+
result = scriptID;
1041+
return true;
1042+
}
1043+
CScript witscript = GetScriptForWitness(subscript);
1044+
pwalletMain->AddCScript(witscript);
1045+
result = CScriptID(witscript);
1046+
return true;
1047+
}
1048+
return false;
1049+
}
1050+
};
1051+
1052+
UniValue addwitnessaddress(const UniValue& params, bool fHelp)
1053+
{
1054+
if (!EnsureWalletIsAvailable(fHelp))
1055+
return NullUniValue;
1056+
1057+
if (fHelp || params.size() < 1 || params.size() > 1)
1058+
{
1059+
string msg = "addwitnessaddress \"address\"\n"
1060+
"\nAdd a witness address for a script (with pubkey or redeemscript known).\n"
1061+
"It returns the witness script.\n"
1062+
1063+
"\nArguments:\n"
1064+
"1. \"address\" (string, required) An address known to the wallet\n"
1065+
1066+
"\nResult:\n"
1067+
"\"witnessaddress\", (string) The value of the new address (P2SH of witness script).\n"
1068+
"}\n"
1069+
;
1070+
throw runtime_error(msg);
1071+
}
1072+
1073+
{
1074+
LOCK(cs_main);
1075+
if (!IsWitnessEnabled(chainActive.Tip(), Params().GetConsensus())) {
1076+
throw JSONRPCError(RPC_WALLET_ERROR, "Segregated witness not enabled on network");
1077+
}
1078+
}
1079+
1080+
CBitcoinAddress address(params[0].get_str());
1081+
if (!address.IsValid())
1082+
throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, "Invalid Bitcoin address");
1083+
1084+
Witnessifier w;
1085+
CTxDestination dest = address.Get();
1086+
bool ret = boost::apply_visitor(w, dest);
1087+
if (!ret) {
1088+
throw JSONRPCError(RPC_WALLET_ERROR, "Public key or redeemscript not known to wallet");
1089+
}
1090+
1091+
return CBitcoinAddress(w.result).ToString();
1092+
}
10141093

10151094
struct tallyitem
10161095
{
@@ -2491,6 +2570,7 @@ static const CRPCCommand commands[] =
24912570
{ "hidden", "resendwallettransactions", &resendwallettransactions, true },
24922571
{ "wallet", "abandontransaction", &abandontransaction, false },
24932572
{ "wallet", "addmultisigaddress", &addmultisigaddress, true },
2573+
{ "wallet", "addwitnessaddress", &addwitnessaddress, true },
24942574
{ "wallet", "backupwallet", &backupwallet, true },
24952575
{ "wallet", "dumpprivkey", &dumpprivkey, true },
24962576
{ "wallet", "dumpwallet", &dumpwallet, true },

0 commit comments

Comments
 (0)