diff options
author | Laurent Navet <laurent.navet@gmail.com> | 2015-07-22 00:08:08 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2015-07-22 00:08:08 -0400 |
commit | d76d99b219e1a233a720775c0451c310d34594e8 (patch) | |
tree | 379e71270b14e8475bbcc8c97c3de71420edaede | |
parent | 5ba92bcf0dd63b35a85e90c3016989733b239bb5 (diff) |
ext4 crypto: exit cleanly if ext4_derive_key_aes() fails
Return value of ext4_derive_key_aes() is stored but not used.
Add test to exit cleanly if ext4_derive_key_aes() fail.
Also fix coverity CID 1309760.
Signed-off-by: Laurent Navet <laurent.navet@gmail.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
-rw-r--r-- | fs/ext4/crypto_key.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/ext4/crypto_key.c b/fs/ext4/crypto_key.c index 442d24e8efc0..ce75bc8b9aef 100644 --- a/fs/ext4/crypto_key.c +++ b/fs/ext4/crypto_key.c | |||
@@ -220,6 +220,8 @@ retry: | |||
220 | BUG_ON(master_key->size != EXT4_AES_256_XTS_KEY_SIZE); | 220 | BUG_ON(master_key->size != EXT4_AES_256_XTS_KEY_SIZE); |
221 | res = ext4_derive_key_aes(ctx.nonce, master_key->raw, | 221 | res = ext4_derive_key_aes(ctx.nonce, master_key->raw, |
222 | raw_key); | 222 | raw_key); |
223 | if (res) | ||
224 | goto out; | ||
223 | got_key: | 225 | got_key: |
224 | ctfm = crypto_alloc_ablkcipher(cipher_str, 0, 0); | 226 | ctfm = crypto_alloc_ablkcipher(cipher_str, 0, 0); |
225 | if (!ctfm || IS_ERR(ctfm)) { | 227 | if (!ctfm || IS_ERR(ctfm)) { |