diff options
author | Theodore Ts'o <tytso@mit.edu> | 2015-12-10 00:57:58 -0500 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2015-12-10 00:57:58 -0500 |
commit | db7730e3091a52c2fcd8fcc952b964d88998e675 (patch) | |
tree | 519a9fefb48bc62866dc0923e408f949220089cf | |
parent | ba5843f51d468644b094674c0317c9ab95632caa (diff) |
ext4 crypto: add missing locking for keyring_key access
Cc: stable@kernel.org
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
-rw-r--r-- | fs/ext4/crypto_key.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/ext4/crypto_key.c b/fs/ext4/crypto_key.c index c5882b36e558..9a16d1e75a49 100644 --- a/fs/ext4/crypto_key.c +++ b/fs/ext4/crypto_key.c | |||
@@ -213,9 +213,11 @@ retry: | |||
213 | res = -ENOKEY; | 213 | res = -ENOKEY; |
214 | goto out; | 214 | goto out; |
215 | } | 215 | } |
216 | down_read(&keyring_key->sem); | ||
216 | ukp = user_key_payload(keyring_key); | 217 | ukp = user_key_payload(keyring_key); |
217 | if (ukp->datalen != sizeof(struct ext4_encryption_key)) { | 218 | if (ukp->datalen != sizeof(struct ext4_encryption_key)) { |
218 | res = -EINVAL; | 219 | res = -EINVAL; |
220 | up_read(&keyring_key->sem); | ||
219 | goto out; | 221 | goto out; |
220 | } | 222 | } |
221 | master_key = (struct ext4_encryption_key *)ukp->data; | 223 | master_key = (struct ext4_encryption_key *)ukp->data; |
@@ -226,10 +228,12 @@ retry: | |||
226 | "ext4: key size incorrect: %d\n", | 228 | "ext4: key size incorrect: %d\n", |
227 | master_key->size); | 229 | master_key->size); |
228 | res = -ENOKEY; | 230 | res = -ENOKEY; |
231 | up_read(&keyring_key->sem); | ||
229 | goto out; | 232 | goto out; |
230 | } | 233 | } |
231 | res = ext4_derive_key_aes(ctx.nonce, master_key->raw, | 234 | res = ext4_derive_key_aes(ctx.nonce, master_key->raw, |
232 | raw_key); | 235 | raw_key); |
236 | up_read(&keyring_key->sem); | ||
233 | if (res) | 237 | if (res) |
234 | goto out; | 238 | goto out; |
235 | got_key: | 239 | got_key: |