diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2019-02-22 20:48:50 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2019-02-22 20:48:50 -0500 |
commit | cb268d806972c76c34e5d74343fb6064cd722c7c (patch) | |
tree | b51d3f5c0431da9b15c65736fe67f9d27e7aca93 /security/keys/keyring.c | |
parent | ef4edb3ed830cbbb443de9906b8cf16dc0653a74 (diff) | |
parent | ede0fa98a900e657d1fcd80b50920efc896c1a4c (diff) |
Merge branch 'fixes-v5.0-rc7' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security
Pull keys fixes from James Morris:
"Two fixes from Eric Biggers"
* 'fixes-v5.0-rc7' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security:
KEYS: always initialize keyring_index_key::desc_len
KEYS: user: Align the payload buffer
Diffstat (limited to 'security/keys/keyring.c')
-rw-r--r-- | security/keys/keyring.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/security/keys/keyring.c b/security/keys/keyring.c index eadebb92986a..f81372f53dd7 100644 --- a/security/keys/keyring.c +++ b/security/keys/keyring.c | |||
@@ -661,9 +661,6 @@ static bool search_nested_keyrings(struct key *keyring, | |||
661 | BUG_ON((ctx->flags & STATE_CHECKS) == 0 || | 661 | BUG_ON((ctx->flags & STATE_CHECKS) == 0 || |
662 | (ctx->flags & STATE_CHECKS) == STATE_CHECKS); | 662 | (ctx->flags & STATE_CHECKS) == STATE_CHECKS); |
663 | 663 | ||
664 | if (ctx->index_key.description) | ||
665 | ctx->index_key.desc_len = strlen(ctx->index_key.description); | ||
666 | |||
667 | /* Check to see if this top-level keyring is what we are looking for | 664 | /* Check to see if this top-level keyring is what we are looking for |
668 | * and whether it is valid or not. | 665 | * and whether it is valid or not. |
669 | */ | 666 | */ |
@@ -914,6 +911,7 @@ key_ref_t keyring_search(key_ref_t keyring, | |||
914 | struct keyring_search_context ctx = { | 911 | struct keyring_search_context ctx = { |
915 | .index_key.type = type, | 912 | .index_key.type = type, |
916 | .index_key.description = description, | 913 | .index_key.description = description, |
914 | .index_key.desc_len = strlen(description), | ||
917 | .cred = current_cred(), | 915 | .cred = current_cred(), |
918 | .match_data.cmp = key_default_cmp, | 916 | .match_data.cmp = key_default_cmp, |
919 | .match_data.raw_data = description, | 917 | .match_data.raw_data = description, |