mirror of
https://gitlab.torproject.org/tpo/core/tor.git
synced 2025-02-24 22:58:50 +01:00
Add some asserts to get_{tlsclient|server}_identity_key
We now require that: - Only actual servers should ever call get_server_identity_key - If you're being a client or bridge, the client and server keys should differ. - If you're being a public relay, the client and server keys should be the same.
This commit is contained in:
parent
2a2301e411
commit
dc557e8164
1 changed files with 21 additions and 1 deletions
|
@ -121,13 +121,32 @@ set_server_identity_key(crypto_pk_env_t *k)
|
|||
crypto_pk_get_digest(server_identitykey, server_identitykey_digest);
|
||||
}
|
||||
|
||||
/** Make sure that we have set up our identity keys to match or not match as
|
||||
* appropriate, and die with an assertion if we have not. */
|
||||
static void
|
||||
assert_identity_keys_ok(void)
|
||||
{
|
||||
tor_assert(client_identitykey);
|
||||
if (public_server_mode(get_options())) {
|
||||
/* assert that we have set the client and server keys to be equal */
|
||||
tor_assert(server_identitykey);
|
||||
tor_assert(0==crypto_pk_cmp_keys(client_identitykey, server_identitykey));
|
||||
} else {
|
||||
/* assert that we have set the client and server keys to be unequal */
|
||||
if (server_identitykey)
|
||||
tor_assert(0!=crypto_pk_cmp_keys(client_identitykey, server_identitykey));
|
||||
}
|
||||
}
|
||||
|
||||
/** Returns the current server identity key; requires that the key has
|
||||
* been set.
|
||||
* been set, and that we are running as a Tor server.
|
||||
*/
|
||||
crypto_pk_env_t *
|
||||
get_server_identity_key(void)
|
||||
{
|
||||
tor_assert(server_identitykey);
|
||||
tor_assert(server_mode(get_options()));
|
||||
assert_identity_keys_ok();
|
||||
return server_identitykey;
|
||||
}
|
||||
|
||||
|
@ -155,6 +174,7 @@ crypto_pk_env_t *
|
|||
get_tlsclient_identity_key(void)
|
||||
{
|
||||
tor_assert(client_identitykey);
|
||||
assert_identity_keys_ok();
|
||||
return client_identitykey;
|
||||
}
|
||||
|
||||
|
|
Loading…
Add table
Reference in a new issue