diff options
Diffstat (limited to 'security')
| -rw-r--r-- | security/keys/gc.c | 3 | ||||
| -rw-r--r-- | security/keys/keyring.c | 4 |
2 files changed, 5 insertions, 2 deletions
diff --git a/security/keys/gc.c b/security/keys/gc.c index 4770be375ffe..19902319d097 100644 --- a/security/keys/gc.c +++ b/security/keys/gc.c | |||
| @@ -77,7 +77,8 @@ static bool key_gc_keyring(struct key *keyring, time_t limit) | |||
| 77 | goto dont_gc; | 77 | goto dont_gc; |
| 78 | 78 | ||
| 79 | /* scan the keyring looking for dead keys */ | 79 | /* scan the keyring looking for dead keys */ |
| 80 | klist = rcu_dereference(keyring->payload.subscriptions); | 80 | klist = rcu_dereference_check(keyring->payload.subscriptions, |
| 81 | lockdep_is_held(&key_serial_lock)); | ||
| 81 | if (!klist) | 82 | if (!klist) |
| 82 | goto dont_gc; | 83 | goto dont_gc; |
| 83 | 84 | ||
diff --git a/security/keys/keyring.c b/security/keys/keyring.c index 8ec02746ca99..e814d2109f8e 100644 --- a/security/keys/keyring.c +++ b/security/keys/keyring.c | |||
| @@ -151,7 +151,9 @@ static void keyring_destroy(struct key *keyring) | |||
| 151 | write_unlock(&keyring_name_lock); | 151 | write_unlock(&keyring_name_lock); |
| 152 | } | 152 | } |
| 153 | 153 | ||
| 154 | klist = rcu_dereference(keyring->payload.subscriptions); | 154 | klist = rcu_dereference_check(keyring->payload.subscriptions, |
| 155 | rcu_read_lock_held() || | ||
| 156 | atomic_read(&keyring->usage) == 0); | ||
| 155 | if (klist) { | 157 | if (klist) { |
| 156 | for (loop = klist->nkeys - 1; loop >= 0; loop--) | 158 | for (loop = klist->nkeys - 1; loop >= 0; loop--) |
| 157 | key_put(klist->keys[loop]); | 159 | key_put(klist->keys[loop]); |
