X-Git-Url: https://git.novaco.in/?p=novacoin.git;a=blobdiff_plain;f=src%2Frpcrawtransaction.cpp;h=b8365c633d525493dfdefc763bd883f408aab5ab;hp=c58170b1111dcc43f3756df4df52bb73f70f8d18;hb=9e58e0a8ca28b15a4bfa677f5b23891972db40fd;hpb=ac7be422c9ecf17e881d07a501b55f31410671da diff --git a/src/rpcrawtransaction.cpp b/src/rpcrawtransaction.cpp index c58170b..b8365c6 100644 --- a/src/rpcrawtransaction.cpp +++ b/src/rpcrawtransaction.cpp @@ -43,17 +43,21 @@ void ScriptPubKeyToJSON(const CScript& scriptPubKey, Object& out, bool fIncludeH if (type == TX_PUBKEY_DROP) { vector vSolutions; - if (!Solver(scriptPubKey, type, vSolutions)) - { - out.push_back(Pair("keyVariant", HexStr(vSolutions[0]))); - out.push_back(Pair("R", HexStr(vSolutions[1]))); - } - } + Solver(scriptPubKey, type, vSolutions); + out.push_back(Pair("keyVariant", HexStr(vSolutions[0]))); + out.push_back(Pair("R", HexStr(vSolutions[1]))); - Array a; - BOOST_FOREACH(const CTxDestination& addr, addresses) - a.push_back(CBitcoinAddress(addr).ToString()); - out.push_back(Pair("addresses", a)); + CMalleableKeyView view; + if (pwalletMain->CheckOwnership(CPubKey(vSolutions[0]), CPubKey(vSolutions[1]), view)) + out.push_back(Pair("pubkeyPair", CBitcoinAddress(view.GetMalleablePubKey()).ToString())); + } + else + { + Array a; + BOOST_FOREACH(const CTxDestination& addr, addresses) + a.push_back(CBitcoinAddress(addr).ToString()); + out.push_back(Pair("addresses", a)); + } } else { @@ -289,23 +293,21 @@ Value createrawtransaction(const Array& params, bool fHelp) // Create output destination script CScript scriptPubKey; CBitcoinAddress address(s.name_); - if (!address.IsValid()) + + if (address.IsValid()) { - CMalleablePubKey mpk(s.name_); - if (!mpk.IsValid()) - throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, string("Invalid output destination: ")+s.name_); + scriptPubKey.SetAddress(address); - CPubKey keyVariant, R; - mpk.GetVariant(R, keyVariant); - scriptPubKey.SetDestination(R, keyVariant); + // Don't perform duplication checking for pubkey-pair addresses + if (!address.IsPair()) + { + if (setAddress.count(address)) + throw JSONRPCError(RPC_INVALID_PARAMETER, string("Invalid parameter, duplicated address: ")+s.name_); + setAddress.insert(address); + } } else - { - scriptPubKey.SetDestination(address.Get()); - if (setAddress.count(address)) - throw JSONRPCError(RPC_INVALID_PARAMETER, string("Invalid parameter, duplicated address: ")+s.name_); - setAddress.insert(address); - } + throw JSONRPCError(RPC_INVALID_ADDRESS_OR_KEY, string("Invalid output destination: ")+s.name_); int64_t nAmount = AmountFromValue(s.value_); @@ -640,7 +642,6 @@ Value createmultisig(const Array& params, bool fHelp) int nRequired = params[0].get_int(); const Array& keys = params[1].get_array(); - string strAccount; // Gather public keys if (nRequired < 1) @@ -651,7 +652,7 @@ Value createmultisig(const Array& params, bool fHelp) "(got %" PRIszu " keys, but need at least %d to redeem)", keys.size(), nRequired)); if (keys.size() > 16) throw runtime_error("Number of addresses involved in the multisignature address creation > 16\nReduce the number"); - std::vector pubkeys; + std::vector pubkeys; pubkeys.resize(keys.size()); for (unsigned int i = 0; i < keys.size(); i++) { @@ -669,16 +670,18 @@ Value createmultisig(const Array& params, bool fHelp) if (!pwalletMain->GetPubKey(keyID, vchPubKey)) throw runtime_error( strprintf("no full public key for address %s",ks.c_str())); - if (!vchPubKey.IsValid() || !pubkeys[i].SetPubKey(vchPubKey)) + if (!vchPubKey.IsFullyValid()) throw runtime_error(" Invalid public key: "+ks); + pubkeys[i] = vchPubKey; } // Case 2: hex public key else if (IsHex(ks)) { CPubKey vchPubKey(ParseHex(ks)); - if (!vchPubKey.IsValid() || !pubkeys[i].SetPubKey(vchPubKey)) + if (!vchPubKey.IsFullyValid()) throw runtime_error(" Invalid public key: "+ks); + pubkeys[i] = vchPubKey; } else {