diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2015-04-23 16:37:46 +0800 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2015-04-26 14:33:16 +0800 |
| commit | 80f7b3552c1c925478a955fd4c700345beaf2982 (patch) | |
| tree | 5652a9f00b827f100a533b843ed336c3584fff55 /crypto/aead.c | |
| parent | 26739535206e819946b0740347c09c94c4e48ba9 (diff) | |
| download | op-kernel-dev-80f7b3552c1c925478a955fd4c700345beaf2982.zip op-kernel-dev-80f7b3552c1c925478a955fd4c700345beaf2982.tar.gz | |
crypto: aead - Fix corner case in crypto_lookup_aead
When the user explicitly states that they don't care whether the
algorithm has been tested (type = CRYPTO_ALG_TESTED and mask = 0),
there is a corner case where we may erroneously return ENOENT.
This patch fixes it by correcting the logic in the test.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'crypto/aead.c')
| -rw-r--r-- | crypto/aead.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/aead.c b/crypto/aead.c index 2222710..d6ad0c6 100644 --- a/crypto/aead.c +++ b/crypto/aead.c @@ -489,7 +489,7 @@ struct crypto_alg *crypto_lookup_aead(const char *name, u32 type, u32 mask) return alg; if (alg->cra_type == &crypto_aead_type) { - if ((alg->cra_flags ^ type ^ ~mask) & CRYPTO_ALG_TESTED) { + if (~alg->cra_flags & (type ^ ~mask) & CRYPTO_ALG_TESTED) { crypto_mod_put(alg); alg = ERR_PTR(-ENOENT); } |
