aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTheodore Ts'o <tytso@mit.edu>2015-12-10 00:57:58 -0500
committerTheodore Ts'o <tytso@mit.edu>2015-12-10 00:57:58 -0500
commitdb7730e3091a52c2fcd8fcc952b964d88998e675 (patch)
tree519a9fefb48bc62866dc0923e408f949220089cf
parentba5843f51d468644b094674c0317c9ab95632caa (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.c4
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;
235got_key: 239got_key: