9pfs/solanum-vs-hackint-and-charybdis
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

mr_server: Report certificate fingerprint mismatches

Browse source 
Log the received certificate fingerprint when it causes a server to be
rejected.
This commit is contained in:
Simon Arlott 2016-04-23 17:32:24 +01:00 committed by Keith Buck
parent 8d0153ff55
commit 598a7d3b7e
2 changed files with 27 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
modules/core/m_server.c
View file

@ -188,6 +188,15 @@ mr_server(struct Client *client_p, struct Client *source_p, int parc, const char
exit_client(client_p, client_p, client_p, "Access denied, requires SSL/TLS but is plaintext");
return 0;
case -6:
sendto_realops_snomask(SNO_GENERAL, L_ALL,
"Connection from servername %s has invalid certificate fingerprint %s",
name, client_p->certfp);
ilog(L_SERVER, "Access denied, invalid certificate fingerprint %s from %s",
client_p->certfp, log_client_name(client_p, SHOW_IP));
exit_client(client_p, client_p, client_p, "Invalid fingerprint.");
return 0;
default:
sendto_realops_snomask(SNO_GENERAL, L_ALL,
"Connection from servername %s rejected, unknown error %d",

21
src/s_serv.c
View file

@ -330,6 +330,9 @@ check_server(const char *name, struct Client *client_p)
rb_dlink_node *ptr;
int error = -1;
const char *encr;
int name_matched = 0;
int host_matched = 0;
int certfp_failed = 0;
s_assert(NULL != client_p);
if(client_p == NULL)
@ -351,14 +354,14 @@ check_server(const char *name, struct Client *client_p)
if(!match(tmp_p->name, name))
continue;
error = -3;
name_matched = 1;
/* XXX: Fix me for IPv6 */
/* XXX sockhost is the IPv4 ip as a string */
if(match(tmp_p->host, client_p->host) ||
match(tmp_p->host, client_p->sockhost))
{
error = -2;
host_matched = 1;
if(tmp_p->passwd)
{
@ -380,8 +383,10 @@ check_server(const char *name, struct Client *client_p)
if(tmp_p->certfp)
{
if(!client_p->certfp || strcasecmp(tmp_p->certfp, client_p->certfp) != 0)
if(!client_p->certfp || strcasecmp(tmp_p->certfp, client_p->certfp) != 0) {
certfp_failed = 1;
continue;
}
}
server_p = tmp_p;
@ -390,7 +395,17 @@ check_server(const char *name, struct Client *client_p)
}
if(server_p == NULL)
{
/* return the most specific error */
if(certfp_failed)
error = -6;
else if(host_matched)
error = -2;
else if(name_matched)
error = -3;
return error;
}
if(ServerConfSSL(server_p) && client_p->localClient->ssl_ctl == NULL)
{