aboutsummaryrefslogtreecommitdiffstats
path: root/security/keys
diff options
context:
space:
mode:
authorDavid Howells <dhowells@redhat.com>2013-08-30 11:07:37 -0400
committerDavid Howells <dhowells@redhat.com>2013-09-25 12:17:01 -0400
commit008643b86c5f33c115c84ccdda1725cac3ad50ad (patch)
tree951ea0d3d7b84ce3570da17f03f45a53f3e4b35d /security/keys
parentb56e5a17b6b9acd16997960504b9940d0d7984e7 (diff)
KEYS: Add a 'trusted' flag and a 'trusted only' flag
Add KEY_FLAG_TRUSTED to indicate that a key either comes from a trusted source or had a cryptographic signature chain that led back to a trusted key the kernel already possessed. Add KEY_FLAGS_TRUSTED_ONLY to indicate that a keyring will only accept links to keys marked with KEY_FLAGS_TRUSTED. Signed-off-by: David Howells <dhowells@redhat.com> Reviewed-by: Kees Cook <keescook@chromium.org>
Diffstat (limited to 'security/keys')
-rw-r--r--security/keys/key.c8
-rw-r--r--security/keys/keyring.c4
2 files changed, 12 insertions, 0 deletions
diff --git a/security/keys/key.c b/security/keys/key.c
index a819b5c7d4ec..d331ea9ef380 100644
--- a/security/keys/key.c
+++ b/security/keys/key.c
@@ -300,6 +300,8 @@ struct key *key_alloc(struct key_type *type, const char *desc,
300 300
301 if (!(flags & KEY_ALLOC_NOT_IN_QUOTA)) 301 if (!(flags & KEY_ALLOC_NOT_IN_QUOTA))
302 key->flags |= 1 << KEY_FLAG_IN_QUOTA; 302 key->flags |= 1 << KEY_FLAG_IN_QUOTA;
303 if (flags & KEY_ALLOC_TRUSTED)
304 key->flags |= 1 << KEY_FLAG_TRUSTED;
303 305
304 memset(&key->type_data, 0, sizeof(key->type_data)); 306 memset(&key->type_data, 0, sizeof(key->type_data));
305 307
@@ -813,6 +815,7 @@ key_ref_t key_create_or_update(key_ref_t keyring_ref,
813 prep.data = payload; 815 prep.data = payload;
814 prep.datalen = plen; 816 prep.datalen = plen;
815 prep.quotalen = index_key.type->def_datalen; 817 prep.quotalen = index_key.type->def_datalen;
818 prep.trusted = flags & KEY_ALLOC_TRUSTED;
816 if (index_key.type->preparse) { 819 if (index_key.type->preparse) {
817 ret = index_key.type->preparse(&prep); 820 ret = index_key.type->preparse(&prep);
818 if (ret < 0) { 821 if (ret < 0) {
@@ -827,6 +830,11 @@ key_ref_t key_create_or_update(key_ref_t keyring_ref,
827 } 830 }
828 index_key.desc_len = strlen(index_key.description); 831 index_key.desc_len = strlen(index_key.description);
829 832
833 key_ref = ERR_PTR(-EPERM);
834 if (!prep.trusted && test_bit(KEY_FLAG_TRUSTED_ONLY, &keyring->flags))
835 goto error_free_prep;
836 flags |= prep.trusted ? KEY_ALLOC_TRUSTED : 0;
837
830 ret = __key_link_begin(keyring, &index_key, &edit); 838 ret = __key_link_begin(keyring, &index_key, &edit);
831 if (ret < 0) { 839 if (ret < 0) {
832 key_ref = ERR_PTR(ret); 840 key_ref = ERR_PTR(ret);
diff --git a/security/keys/keyring.c b/security/keys/keyring.c
index f7cdea22214f..9b6f6e09b50c 100644
--- a/security/keys/keyring.c
+++ b/security/keys/keyring.c
@@ -1183,6 +1183,10 @@ int key_link(struct key *keyring, struct key *key)
1183 key_check(keyring); 1183 key_check(keyring);
1184 key_check(key); 1184 key_check(key);
1185 1185
1186 if (test_bit(KEY_FLAG_TRUSTED_ONLY, &keyring->flags) &&
1187 !test_bit(KEY_FLAG_TRUSTED, &key->flags))
1188 return -EPERM;
1189
1186 ret = __key_link_begin(keyring, &key->index_key, &edit); 1190 ret = __key_link_begin(keyring, &key->index_key, &edit);
1187 if (ret == 0) { 1191 if (ret == 0) {
1188 kdebug("begun {%d,%d}", keyring->serial, atomic_read(&keyring->usage)); 1192 kdebug("begun {%d,%d}", keyring->serial, atomic_read(&keyring->usage));