Skip to content

Commit da6a668

Browse files
horiagherbertx
authored andcommitted
crypto: caam - silence .setkey in case of bad key length
In case of bad key length, driver emits "key size mismatch" messages, but only for xts(aes) algorithms. Reduce verbosity by making them visible only when debugging. This way crypto fuzz testing log cleans up a bit. Signed-off-by: Horia Geantă <[email protected]> Signed-off-by: Herbert Xu <[email protected]>
1 parent b7ec41d commit da6a668

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

drivers/crypto/caam/caamalg.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -832,7 +832,7 @@ static int xts_skcipher_setkey(struct crypto_skcipher *skcipher, const u8 *key,
832832
u32 *desc;
833833

834834
if (keylen != 2 * AES_MIN_KEY_SIZE && keylen != 2 * AES_MAX_KEY_SIZE) {
835-
dev_err(jrdev, "key size mismatch\n");
835+
dev_dbg(jrdev, "key size mismatch\n");
836836
return -EINVAL;
837837
}
838838

drivers/crypto/caam/caamalg_qi.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -728,7 +728,7 @@ static int xts_skcipher_setkey(struct crypto_skcipher *skcipher, const u8 *key,
728728
int ret = 0;
729729

730730
if (keylen != 2 * AES_MIN_KEY_SIZE && keylen != 2 * AES_MAX_KEY_SIZE) {
731-
dev_err(jrdev, "key size mismatch\n");
731+
dev_dbg(jrdev, "key size mismatch\n");
732732
return -EINVAL;
733733
}
734734

drivers/crypto/caam/caamalg_qi2.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1058,7 +1058,7 @@ static int xts_skcipher_setkey(struct crypto_skcipher *skcipher, const u8 *key,
10581058
u32 *desc;
10591059

10601060
if (keylen != 2 * AES_MIN_KEY_SIZE && keylen != 2 * AES_MAX_KEY_SIZE) {
1061-
dev_err(dev, "key size mismatch\n");
1061+
dev_dbg(dev, "key size mismatch\n");
10621062
return -EINVAL;
10631063
}
10641064

0 commit comments

Comments
 (0)