mirror of
https://github.com/matrix-construct/construct
synced 2024-11-18 07:50:57 +01:00
m_sasl: rate-limit SASL REAUTH usage
This commit is contained in:
parent
834579cecd
commit
9d07a42d7a
2 changed files with 20 additions and 4 deletions
|
@ -285,7 +285,10 @@ struct LocalUser
|
||||||
char sasl_agent[IDLEN];
|
char sasl_agent[IDLEN];
|
||||||
unsigned char sasl_out;
|
unsigned char sasl_out;
|
||||||
unsigned char sasl_complete;
|
unsigned char sasl_complete;
|
||||||
unsigned short sasl_messages;
|
|
||||||
|
unsigned int sasl_messages;
|
||||||
|
unsigned int sasl_failures;
|
||||||
|
time_t sasl_next_retry;
|
||||||
};
|
};
|
||||||
|
|
||||||
struct AuthClient
|
struct AuthClient
|
||||||
|
|
|
@ -131,14 +131,20 @@ m_authenticate(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *
|
||||||
if(!IsCapable(source_p, CLICAP_SASL))
|
if(!IsCapable(source_p, CLICAP_SASL))
|
||||||
return;
|
return;
|
||||||
|
|
||||||
if (strlen(client_p->id) == 3)
|
if(source_p->localClient->sasl_next_retry > rb_current_time())
|
||||||
|
{
|
||||||
|
sendto_one(source_p, form_str(RPL_LOAD2HI), me.name, EmptyString(source_p->name) ? "*" : source_p->name, msgbuf_p->cmd);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if(strlen(client_p->id) == 3)
|
||||||
{
|
{
|
||||||
exit_client(client_p, client_p, client_p, "Mixing client and server protocol");
|
exit_client(client_p, client_p, client_p, "Mixing client and server protocol");
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
saslserv_p = find_named_client(ConfigFileEntry.sasl_service);
|
saslserv_p = find_named_client(ConfigFileEntry.sasl_service);
|
||||||
if (saslserv_p == NULL || !IsService(saslserv_p))
|
if(saslserv_p == NULL || !IsService(saslserv_p))
|
||||||
{
|
{
|
||||||
sendto_one(source_p, form_str(ERR_SASLABORTED), me.name, EmptyString(source_p->name) ? "*" : source_p->name);
|
sendto_one(source_p, form_str(ERR_SASLABORTED), me.name, EmptyString(source_p->name) ? "*" : source_p->name);
|
||||||
return;
|
return;
|
||||||
|
@ -233,9 +239,15 @@ me_sasl(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_
|
||||||
if(*parv[4] == 'F')
|
if(*parv[4] == 'F')
|
||||||
{
|
{
|
||||||
sendto_one(target_p, form_str(ERR_SASLFAIL), me.name, EmptyString(target_p->name) ? "*" : target_p->name);
|
sendto_one(target_p, form_str(ERR_SASLFAIL), me.name, EmptyString(target_p->name) ? "*" : target_p->name);
|
||||||
|
/* Failures with zero messages are just "unknown mechanism" errors; don't count those. */
|
||||||
if(target_p->localClient->sasl_messages > 0)
|
if(target_p->localClient->sasl_messages > 0)
|
||||||
{
|
{
|
||||||
if(throttle_add((struct sockaddr*)&target_p->localClient->ip))
|
if(*target_p->name)
|
||||||
|
{
|
||||||
|
target_p->localClient->sasl_failures++;
|
||||||
|
target_p->localClient->sasl_next_retry = rb_current_time() + (1 << MIN(target_p->localClient->sasl_failures + 5, 13));
|
||||||
|
}
|
||||||
|
else if(throttle_add((struct sockaddr*)&target_p->localClient->ip))
|
||||||
{
|
{
|
||||||
exit_client(target_p, target_p, &me, "Too many failed authentication attempts");
|
exit_client(target_p, target_p, &me, "Too many failed authentication attempts");
|
||||||
return;
|
return;
|
||||||
|
@ -245,6 +257,7 @@ me_sasl(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_
|
||||||
else if(*parv[4] == 'S')
|
else if(*parv[4] == 'S')
|
||||||
{
|
{
|
||||||
sendto_one(target_p, form_str(RPL_SASLSUCCESS), me.name, EmptyString(target_p->name) ? "*" : target_p->name);
|
sendto_one(target_p, form_str(RPL_SASLSUCCESS), me.name, EmptyString(target_p->name) ? "*" : target_p->name);
|
||||||
|
target_p->localClient->sasl_failures = 0;
|
||||||
target_p->localClient->sasl_complete = 1;
|
target_p->localClient->sasl_complete = 1;
|
||||||
ServerStats.is_ssuc++;
|
ServerStats.is_ssuc++;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue