KEYS: fix keyctl_set_reqkey_keyring() to not leak thread keyrings
authorEric Biggers <ebiggers@google.com>
Tue, 18 Apr 2017 14:31:09 +0000 (15:31 +0100)
committerWilly Tarreau <w@1wt.eu>
Wed, 7 Jun 2017 22:46:48 +0000 (00:46 +0200)
commitd19182e28cfd7ade11a71f1e8190fb7243ed4bf5
tree4b774698bf8122090656563d5cb413a42ca88077
parent5b22a8a5ba91e92b554d4799eea3bbfef3bc0af4
KEYS: fix keyctl_set_reqkey_keyring() to not leak thread keyrings

commit c9f838d104fed6f2f61d68164712e3204bf5271b upstream.

This fixes CVE-2017-7472.

Running the following program as an unprivileged user exhausts kernel
memory by leaking thread keyrings:

#include <keyutils.h>

int main()
{
for (;;)
keyctl_set_reqkey_keyring(KEY_REQKEY_DEFL_THREAD_KEYRING);
}

Fix it by only creating a new thread keyring if there wasn't one before.
To make things more consistent, make install_thread_keyring_to_cred()
and install_process_keyring_to_cred() both return 0 if the corresponding
keyring is already present.

Fixes: d84f4f992cbd ("CRED: Inaugurate COW credentials")
Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: David Howells <dhowells@redhat.com>
Signed-off-by: Willy Tarreau <w@1wt.eu>
security/keys/keyctl.c
security/keys/process_keys.c