diff options
| author | Milan Broz <gmazyland@gmail.com> | 2018-02-13 14:50:50 +0100 |
|---|---|---|
| committer | Mike Snitzer <snitzer@redhat.com> | 2018-04-03 21:04:16 +0200 |
| commit | e16b4f99f0f79682a7efe191a8ce694d87ca9fc8 (patch) | |
| tree | 8176b07c744f1d532d629cb9a5b20159f16357cd /drivers/md | |
| parent | dm: remove unused macro DM_MOD_NAME_SIZE (diff) | |
| download | linux-e16b4f99f0f79682a7efe191a8ce694d87ca9fc8.tar.xz linux-e16b4f99f0f79682a7efe191a8ce694d87ca9fc8.zip | |
dm integrity: fail early if required HMAC key is not available
Since crypto API commit 9fa68f62004 ("crypto: hash - prevent using keyed
hashes without setting key") dm-integrity cannot use keyed algorithms
without the key being set.
The dm-integrity recognizes this too late (during use of HMAC), so it
allows creation and formatting of superblock, but the device is in fact
unusable.
Fix it by detecting the key requirement in integrity table constructor.
Signed-off-by: Milan Broz <gmazyland@gmail.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Diffstat (limited to 'drivers/md')
| -rw-r--r-- | drivers/md/dm-integrity.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/md/dm-integrity.c b/drivers/md/dm-integrity.c index 46d7c8749222..6c81b11d0521 100644 --- a/drivers/md/dm-integrity.c +++ b/drivers/md/dm-integrity.c @@ -2548,6 +2548,9 @@ static int get_mac(struct crypto_shash **hash, struct alg_spec *a, char **error, *error = error_key; return r; } + } else if (crypto_shash_get_flags(*hash) & CRYPTO_TFM_NEED_KEY) { + *error = error_key; + return -ENOKEY; } } |