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

authd: fix reference far off the end of the array on shutdown

Browse source 
When authd has no more input it tries to reject all current clients with
an id of UINT32_MAX.
This commit is contained in:
Simon Arlott 2016-05-01 11:50:26 +01:00
parent 2f598dacdd
commit 84d0b55e76
No known key found for this signature in database
GPG key ID: C8975F2043CA5D24
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
authd/provider.c
View file

@ -204,8 +204,7 @@ cancel_providers(struct auth_client *auth)
}
}
/* Provider is done
* WARNING: do not use auth instance after calling! */
/* Provider is done */
void
provider_done(struct auth_client *auth, uint32_t id)
{
@ -234,8 +233,7 @@ provider_done(struct auth_client *auth, uint32_t id)
}
}
/* Reject a client and cancel any outstanding providers
* WARNING: do not use auth instance after calling! */
/* Reject a client and cancel any outstanding providers */
void
reject_client(struct auth_client *auth, uint32_t id, const char *data, const char *fmt, ...)
{
@ -251,7 +249,7 @@ reject_client(struct auth_client *auth, uint32_t id, const char *data, const cha
* --Elizafox
*/
rb_helper_write(authd_helper, "R %x %c %s %s %s :%s",
auth->cid, auth->data[id].provider->letter,
auth->cid, id != UINT32_MAX ? auth->data[id].provider->letter : '*',
auth->username, auth->hostname,
data == NULL ? "*" : data, buf);