Merge pull request #6600

fa06c39 Bind signature to full address and signing mode (SarangNoether)
743608e wallet: allow signing a message with spend or view key (moneromooo-monero)
This commit is contained in:
luigi1111 2020-08-31 16:49:03 -05:00
commit 61dd04b681
No known key found for this signature in database
GPG Key ID: F4ACA0183641E010
11 changed files with 186 additions and 55 deletions

View File

@ -230,6 +230,7 @@ namespace config
const unsigned char HASH_KEY_CLSAG_ROUND[] = "CLSAG_round"; const unsigned char HASH_KEY_CLSAG_ROUND[] = "CLSAG_round";
const unsigned char HASH_KEY_CLSAG_AGG_0[] = "CLSAG_agg_0"; const unsigned char HASH_KEY_CLSAG_AGG_0[] = "CLSAG_agg_0";
const unsigned char HASH_KEY_CLSAG_AGG_1[] = "CLSAG_agg_1"; const unsigned char HASH_KEY_CLSAG_AGG_1[] = "CLSAG_agg_1";
const char HASH_KEY_MESSAGE_SIGNING[] = "MoneroMessageSignature";
namespace testnet namespace testnet
{ {

View File

@ -223,7 +223,7 @@ namespace
const char* USAGE_GET_TX_NOTE("get_tx_note <txid>"); const char* USAGE_GET_TX_NOTE("get_tx_note <txid>");
const char* USAGE_GET_DESCRIPTION("get_description"); const char* USAGE_GET_DESCRIPTION("get_description");
const char* USAGE_SET_DESCRIPTION("set_description [free text note]"); const char* USAGE_SET_DESCRIPTION("set_description [free text note]");
const char* USAGE_SIGN("sign [<account_index>,<address_index>] <filename>"); const char* USAGE_SIGN("sign [<account_index>,<address_index>] [--spend|--view] <filename>");
const char* USAGE_VERIFY("verify <filename> <address> <signature>"); const char* USAGE_VERIFY("verify <filename> <address> <signature>");
const char* USAGE_EXPORT_KEY_IMAGES("export_key_images [all] <filename>"); const char* USAGE_EXPORT_KEY_IMAGES("export_key_images [all] <filename>");
const char* USAGE_IMPORT_KEY_IMAGES("import_key_images <filename>"); const char* USAGE_IMPORT_KEY_IMAGES("import_key_images <filename>");
@ -9877,7 +9877,7 @@ bool simple_wallet::sign(const std::vector<std::string> &args)
fail_msg_writer() << tr("command not supported by HW wallet"); fail_msg_writer() << tr("command not supported by HW wallet");
return true; return true;
} }
if (args.size() != 1 && args.size() != 2) if (args.size() != 1 && args.size() != 2 && args.size() != 3)
{ {
PRINT_USAGE(USAGE_SIGN); PRINT_USAGE(USAGE_SIGN);
return true; return true;
@ -9893,17 +9893,29 @@ bool simple_wallet::sign(const std::vector<std::string> &args)
return true; return true;
} }
tools::wallet2::message_signature_type_t message_signature_type = tools::wallet2::sign_with_spend_key;
subaddress_index index{0, 0}; subaddress_index index{0, 0};
if (args.size() == 2) for (unsigned int idx = 0; idx + 1 < args.size(); ++idx)
{ {
unsigned int a, b; unsigned int a, b;
if (sscanf(args[0].c_str(), "%u,%u", &a, &b) != 2) if (sscanf(args[idx].c_str(), "%u,%u", &a, &b) == 2)
{ {
fail_msg_writer() << tr("Invalid subaddress index format"); index.major = a;
index.minor = b;
}
else if (args[idx] == "--spend")
{
message_signature_type = tools::wallet2::sign_with_spend_key;
}
else if (args[idx] == "--view")
{
message_signature_type = tools::wallet2::sign_with_view_key;
}
else
{
fail_msg_writer() << tr("Invalid subaddress index format, and not a signature type: ") << args[idx];
return true; return true;
} }
index.major = a;
index.minor = b;
} }
const std::string &filename = args.back(); const std::string &filename = args.back();
@ -9917,7 +9929,7 @@ bool simple_wallet::sign(const std::vector<std::string> &args)
SCOPED_WALLET_UNLOCK(); SCOPED_WALLET_UNLOCK();
std::string signature = m_wallet->sign(data, index); std::string signature = m_wallet->sign(data, message_signature_type, index);
success_msg_writer() << signature; success_msg_writer() << signature;
return true; return true;
} }
@ -9948,14 +9960,14 @@ bool simple_wallet::verify(const std::vector<std::string> &args)
return true; return true;
} }
r = m_wallet->verify(data, info.address, signature); tools::wallet2::message_signature_result_t result = m_wallet->verify(data, info.address, signature);
if (!r) if (!result.valid)
{ {
fail_msg_writer() << tr("Bad signature from ") << address_string; fail_msg_writer() << tr("Bad signature from ") << address_string;
} }
else else
{ {
success_msg_writer() << tr("Good signature from ") << address_string; success_msg_writer() << tr("Good signature from ") << address_string << (result.old ? " (using old signature algorithm)" : "") << " with " << (result.type == tools::wallet2::sign_with_spend_key ? "spend key" : result.type == tools::wallet2::sign_with_view_key ? "view key" : "unknown key combination (suspicious)");
} }
return true; return true;
} }

View File

@ -1998,7 +1998,7 @@ bool WalletImpl::checkReserveProof(const std::string &address, const std::string
std::string WalletImpl::signMessage(const std::string &message) std::string WalletImpl::signMessage(const std::string &message)
{ {
return m_wallet->sign(message); return m_wallet->sign(message, tools::wallet2::sign_with_spend_key);
} }
bool WalletImpl::verifySignedMessage(const std::string &message, const std::string &address, const std::string &signature) const bool WalletImpl::verifySignedMessage(const std::string &message, const std::string &address, const std::string &signature) const
@ -2008,7 +2008,7 @@ bool WalletImpl::verifySignedMessage(const std::string &message, const std::stri
if (!cryptonote::get_account_address_from_str(info, m_wallet->nettype(), address)) if (!cryptonote::get_account_address_from_str(info, m_wallet->nettype(), address))
return false; return false;
return m_wallet->verify(message, info.address, signature); return m_wallet->verify(message, info.address, signature).valid;
} }
std::string WalletImpl::signMultisigParticipant(const std::string &message) const std::string WalletImpl::signMultisigParticipant(const std::string &message) const

View File

@ -12207,51 +12207,130 @@ void wallet2::set_account_tag_description(const std::string& tag, const std::str
m_account_tags.first[tag] = description; m_account_tags.first[tag] = description;
} }
std::string wallet2::sign(const std::string &data, cryptonote::subaddress_index index) const // Set up an address signature message hash
// Hash data: domain separator, spend public key, view public key, mode identifier, payload data
static crypto::hash get_message_hash(const std::string &data, const crypto::public_key &spend_key, const crypto::public_key &view_key, const uint8_t mode)
{ {
KECCAK_CTX ctx;
keccak_init(&ctx);
keccak_update(&ctx, (const uint8_t*)config::HASH_KEY_MESSAGE_SIGNING, sizeof(config::HASH_KEY_MESSAGE_SIGNING)); // includes NUL
keccak_update(&ctx, (const uint8_t*)&spend_key, sizeof(crypto::public_key));
keccak_update(&ctx, (const uint8_t*)&view_key, sizeof(crypto::public_key));
keccak_update(&ctx, (const uint8_t*)&mode, sizeof(uint8_t));
char len_buf[(sizeof(size_t) * 8 + 6) / 7];
char *ptr = len_buf;
tools::write_varint(ptr, data.size());
CHECK_AND_ASSERT_THROW_MES(ptr > len_buf && ptr <= len_buf + sizeof(len_buf), "Length overflow");
keccak_update(&ctx, (const uint8_t*)len_buf, ptr - len_buf);
keccak_update(&ctx, (const uint8_t*)data.data(), data.size());
crypto::hash hash; crypto::hash hash;
crypto::cn_fast_hash(data.data(), data.size(), hash); keccak_finish(&ctx, (uint8_t*)&hash);
const cryptonote::account_keys &keys = m_account.get_keys(); return hash;
crypto::signature signature;
crypto::secret_key skey;
crypto::public_key pkey;
if (index.is_zero())
{
skey = keys.m_spend_secret_key;
pkey = keys.m_account_address.m_spend_public_key;
}
else
{
skey = keys.m_spend_secret_key;
crypto::secret_key m = m_account.get_device().get_subaddress_secret_key(keys.m_view_secret_key, index);
sc_add((unsigned char*)&skey, (unsigned char*)&m, (unsigned char*)&skey);
secret_key_to_public_key(skey, pkey);
}
crypto::generate_signature(hash, pkey, skey, signature);
return std::string("SigV1") + tools::base58::encode(std::string((const char *)&signature, sizeof(signature)));
} }
bool wallet2::verify(const std::string &data, const cryptonote::account_public_address &address, const std::string &signature) const // Sign a message with a private key from either the base address or a subaddress
// The signature is also bound to both keys and the signature mode (spend, view) to prevent unintended reuse
std::string wallet2::sign(const std::string &data, message_signature_type_t signature_type, cryptonote::subaddress_index index) const
{ {
const size_t header_len = strlen("SigV1"); const cryptonote::account_keys &keys = m_account.get_keys();
if (signature.size() < header_len || signature.substr(0, header_len) != "SigV1") { crypto::signature signature;
crypto::secret_key skey, m;
crypto::secret_key skey_spend, skey_view;
crypto::public_key pkey;
crypto::public_key pkey_spend, pkey_view; // to include both in hash
crypto::hash hash;
uint8_t mode;
// Use the base address
if (index.is_zero())
{
switch (signature_type)
{
case sign_with_spend_key:
skey = keys.m_spend_secret_key;
pkey = keys.m_account_address.m_spend_public_key;
mode = 0;
break;
case sign_with_view_key:
skey = keys.m_view_secret_key;
pkey = keys.m_account_address.m_view_public_key;
mode = 1;
break;
default: CHECK_AND_ASSERT_THROW_MES(false, "Invalid signature type requested");
}
hash = get_message_hash(data,keys.m_account_address.m_spend_public_key,keys.m_account_address.m_view_public_key,mode);
}
// Use a subaddress
else
{
skey_spend = keys.m_spend_secret_key;
m = m_account.get_device().get_subaddress_secret_key(keys.m_view_secret_key, index);
sc_add((unsigned char*)&skey_spend, (unsigned char*)&m, (unsigned char*)&skey_spend);
secret_key_to_public_key(skey_spend,pkey_spend);
sc_mul((unsigned char*)&skey_view, (unsigned char*)&keys.m_view_secret_key, (unsigned char*)&skey_spend);
secret_key_to_public_key(skey_view,pkey_view);
switch (signature_type)
{
case sign_with_spend_key:
skey = skey_spend;
pkey = pkey_spend;
mode = 0;
break;
case sign_with_view_key:
skey = skey_view;
pkey = pkey_view;
mode = 1;
break;
default: CHECK_AND_ASSERT_THROW_MES(false, "Invalid signature type requested");
}
secret_key_to_public_key(skey, pkey);
hash = get_message_hash(data,pkey_spend,pkey_view,mode);
}
crypto::generate_signature(hash, pkey, skey, signature);
return std::string("SigV2") + tools::base58::encode(std::string((const char *)&signature, sizeof(signature)));
}
tools::wallet2::message_signature_result_t wallet2::verify(const std::string &data, const cryptonote::account_public_address &address, const std::string &signature) const
{
static const size_t v1_header_len = strlen("SigV1");
static const size_t v2_header_len = strlen("SigV2");
const bool v1 = signature.size() >= v1_header_len && signature.substr(0, v1_header_len) == "SigV1";
const bool v2 = signature.size() >= v2_header_len && signature.substr(0, v2_header_len) == "SigV2";
if (!v1 && !v2)
{
LOG_PRINT_L0("Signature header check error"); LOG_PRINT_L0("Signature header check error");
return false; return {};
} }
crypto::hash hash; crypto::hash hash;
crypto::cn_fast_hash(data.data(), data.size(), hash); if (v1)
{
crypto::cn_fast_hash(data.data(), data.size(), hash);
}
std::string decoded; std::string decoded;
if (!tools::base58::decode(signature.substr(header_len), decoded)) { if (!tools::base58::decode(signature.substr(v1 ? v1_header_len : v2_header_len), decoded)) {
LOG_PRINT_L0("Signature decoding error"); LOG_PRINT_L0("Signature decoding error");
return false; return {};
} }
crypto::signature s; crypto::signature s;
if (sizeof(s) != decoded.size()) { if (sizeof(s) != decoded.size()) {
LOG_PRINT_L0("Signature decoding error"); LOG_PRINT_L0("Signature decoding error");
return false; return {};
} }
memcpy(&s, decoded.data(), sizeof(s)); memcpy(&s, decoded.data(), sizeof(s));
return crypto::check_signature(hash, address.m_spend_public_key, s);
// Test each mode and return which mode, if either, succeeded
if (v2)
hash = get_message_hash(data,address.m_spend_public_key,address.m_view_public_key,(uint8_t) 0);
if (crypto::check_signature(hash, address.m_spend_public_key, s))
return {true, v1 ? 1u : 2u, !v2, sign_with_spend_key };
if (v2)
hash = get_message_hash(data,address.m_spend_public_key,address.m_view_public_key,(uint8_t) 1);
if (crypto::check_signature(hash, address.m_view_public_key, s))
return {true, v1 ? 1u : 2u, !v2, sign_with_view_key };
// Both modes failed
return {};
} }
std::string wallet2::sign_multisig_participant(const std::string& data) const std::string wallet2::sign_multisig_participant(const std::string& data) const

View File

@ -1341,8 +1341,10 @@ private:
*/ */
void set_account_tag_description(const std::string& tag, const std::string& description); void set_account_tag_description(const std::string& tag, const std::string& description);
std::string sign(const std::string &data, cryptonote::subaddress_index index = {0, 0}) const; enum message_signature_type_t { sign_with_spend_key, sign_with_view_key };
bool verify(const std::string &data, const cryptonote::account_public_address &address, const std::string &signature) const; std::string sign(const std::string &data, message_signature_type_t signature_type, cryptonote::subaddress_index index = {0, 0}) const;
struct message_signature_result_t { bool valid; unsigned version; bool old; message_signature_type_t type; };
message_signature_result_t verify(const std::string &data, const cryptonote::account_public_address &address, const std::string &signature) const;
/*! /*!
* \brief sign_multisig_participant signs given message with the multisig public signer key * \brief sign_multisig_participant signs given message with the multisig public signer key

View File

@ -2007,7 +2007,18 @@ namespace tools
return false; return false;
} }
res.signature = m_wallet->sign(req.data, {req.account_index, req.address_index}); tools::wallet2::message_signature_type_t signature_type = tools::wallet2::sign_with_spend_key;
if (req.signature_type == "spend" || req.signature_type == "")
signature_type = tools::wallet2::sign_with_spend_key;
else if (req.signature_type == "view")
signature_type = tools::wallet2::sign_with_view_key;
else
{
er.code = WALLET_RPC_ERROR_CODE_INVALID_SIGNATURE_TYPE;
er.message = "Invalid signature type requested";
return false;
}
res.signature = m_wallet->sign(req.data, signature_type, {req.account_index, req.address_index});
return true; return true;
} }
//------------------------------------------------------------------------------------------------------------------------------ //------------------------------------------------------------------------------------------------------------------------------
@ -2042,7 +2053,16 @@ namespace tools
return false; return false;
} }
res.good = m_wallet->verify(req.data, info.address, req.signature); const auto result = m_wallet->verify(req.data, info.address, req.signature);
res.good = result.valid;
res.version = result.version;
res.old = result.old;
switch (result.type)
{
case tools::wallet2::sign_with_spend_key: res.signature_type = "spend"; break;
case tools::wallet2::sign_with_view_key: res.signature_type = "view"; break;
default: res.signature_type = "invalid"; break;
}
return true; return true;
} }
//------------------------------------------------------------------------------------------------------------------------------ //------------------------------------------------------------------------------------------------------------------------------

View File

@ -47,7 +47,7 @@
// advance which version they will stop working with // advance which version they will stop working with
// Don't go over 32767 for any of these // Don't go over 32767 for any of these
#define WALLET_RPC_VERSION_MAJOR 1 #define WALLET_RPC_VERSION_MAJOR 1
#define WALLET_RPC_VERSION_MINOR 19 #define WALLET_RPC_VERSION_MINOR 20
#define MAKE_WALLET_RPC_VERSION(major,minor) (((major)<<16)|(minor)) #define MAKE_WALLET_RPC_VERSION(major,minor) (((major)<<16)|(minor))
#define WALLET_RPC_VERSION MAKE_WALLET_RPC_VERSION(WALLET_RPC_VERSION_MAJOR, WALLET_RPC_VERSION_MINOR) #define WALLET_RPC_VERSION MAKE_WALLET_RPC_VERSION(WALLET_RPC_VERSION_MAJOR, WALLET_RPC_VERSION_MINOR)
namespace tools namespace tools
@ -1618,11 +1618,13 @@ namespace wallet_rpc
std::string data; std::string data;
uint32_t account_index; uint32_t account_index;
uint32_t address_index; uint32_t address_index;
std::string signature_type;
BEGIN_KV_SERIALIZE_MAP() BEGIN_KV_SERIALIZE_MAP()
KV_SERIALIZE(data) KV_SERIALIZE(data)
KV_SERIALIZE_OPT(account_index, 0u) KV_SERIALIZE_OPT(account_index, 0u)
KV_SERIALIZE_OPT(address_index, 0u) KV_SERIALIZE_OPT(address_index, 0u)
KV_SERIALIZE(signature_type)
END_KV_SERIALIZE_MAP() END_KV_SERIALIZE_MAP()
}; };
typedef epee::misc_utils::struct_init<request_t> request; typedef epee::misc_utils::struct_init<request_t> request;
@ -1657,9 +1659,15 @@ namespace wallet_rpc
struct response_t struct response_t
{ {
bool good; bool good;
unsigned version;
bool old;
std::string signature_type;
BEGIN_KV_SERIALIZE_MAP() BEGIN_KV_SERIALIZE_MAP()
KV_SERIALIZE(good); KV_SERIALIZE(good);
KV_SERIALIZE(version);
KV_SERIALIZE(old);
KV_SERIALIZE(signature_type);
END_KV_SERIALIZE_MAP() END_KV_SERIALIZE_MAP()
}; };
typedef epee::misc_utils::struct_init<response_t> response; typedef epee::misc_utils::struct_init<response_t> response;

View File

@ -76,3 +76,4 @@
#define WALLET_RPC_ERROR_CODE_NON_DETERMINISTIC -43 #define WALLET_RPC_ERROR_CODE_NON_DETERMINISTIC -43
#define WALLET_RPC_ERROR_CODE_INVALID_LOG_LEVEL -44 #define WALLET_RPC_ERROR_CODE_INVALID_LOG_LEVEL -44
#define WALLET_RPC_ERROR_CODE_ATTRIBUTE_NOT_FOUND -45 #define WALLET_RPC_ERROR_CODE_ATTRIBUTE_NOT_FOUND -45
#define WALLET_RPC_ERROR_CODE_INVALID_SIGNATURE_TYPE -47

View File

@ -43,8 +43,10 @@ from framework.wallet import Wallet
class MessageSigningTest(): class MessageSigningTest():
def run_test(self): def run_test(self):
self.create() self.create()
self.check_signing(False) self.check_signing(False, False)
self.check_signing(True) self.check_signing(False, True)
self.check_signing(True, False)
self.check_signing(True, True)
def create(self): def create(self):
print('Creating wallets') print('Creating wallets')
@ -66,8 +68,8 @@ class MessageSigningTest():
assert res.address == self.address[i] assert res.address == self.address[i]
assert res.seed == seeds[i] assert res.seed == seeds[i]
def check_signing(self, subaddress): def check_signing(self, subaddress, spend_key):
print('Signing/verifing messages with ' + ('subaddress' if subaddress else 'standard address')) print('Signing/verifing messages with ' + ('subaddress' if subaddress else 'standard address') + ' ' + ('spend key' if spend_key else 'view key'))
messages = ['foo', ''] messages = ['foo', '']
if subaddress: if subaddress:
address = [] address = []
@ -84,17 +86,22 @@ class MessageSigningTest():
account_index = 0 account_index = 0
address_index = 0 address_index = 0
for message in messages: for message in messages:
res = self.wallet[0].sign(message, account_index = account_index, address_index = address_index) res = self.wallet[0].sign(message, account_index = account_index, address_index = address_index, signature_type = 'spend' if spend_key else 'view')
signature = res.signature signature = res.signature
for i in range(2): for i in range(2):
res = self.wallet[i].verify(message, address[0], signature) res = self.wallet[i].verify(message, address[0], signature)
assert res.good assert res.good
assert not res.old
assert res.version == 2
assert res.signature_type == 'spend' if spend_key else 'view'
res = self.wallet[i].verify('different', address[0], signature) res = self.wallet[i].verify('different', address[0], signature)
assert not res.good assert not res.good
res = self.wallet[i].verify(message, address[1], signature) res = self.wallet[i].verify(message, address[1], signature)
assert not res.good assert not res.good
res = self.wallet[i].verify(message, address[0], signature + 'x') res = self.wallet[i].verify(message, address[0], signature + 'x')
assert not res.good assert not res.good
res = self.wallet[i].verify(message, address[0], signature.replace('SigV2','SigV1'))
assert not res.good
if __name__ == '__main__': if __name__ == '__main__':
MessageSigningTest().run_test() MessageSigningTest().run_test()

View File

@ -59,6 +59,6 @@ BEGIN_INIT_SIMPLE_FUZZER()
END_INIT_SIMPLE_FUZZER() END_INIT_SIMPLE_FUZZER()
BEGIN_SIMPLE_FUZZER() BEGIN_SIMPLE_FUZZER()
bool valid = wallet->verify("test", address, std::string((const char*)buf, len)); tools::wallet2::message_signature_result_t result = wallet->verify("test", address, s);
std::cout << "Signature " << (valid ? "valid" : "invalid") << std::endl; std::cout << "Signature " << (result.valid ? "valid" : "invalid") << std::endl;
END_SIMPLE_FUZZER() END_SIMPLE_FUZZER()

View File

@ -706,13 +706,14 @@ class Wallet(object):
} }
return self.rpc.send_json_rpc_request(check_reserve_proof) return self.rpc.send_json_rpc_request(check_reserve_proof)
def sign(self, data, account_index = 0, address_index = 0): def sign(self, data, account_index = 0, address_index = 0, signature_type = ""):
sign = { sign = {
'method': 'sign', 'method': 'sign',
'params' : { 'params' : {
'data': data, 'data': data,
'account_index': account_index, 'account_index': account_index,
'address_index': address_index, 'address_index': address_index,
'signature_type': signature_type,
}, },
'jsonrpc': '2.0', 'jsonrpc': '2.0',
'id': '0' 'id': '0'