diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-05-03 11:08:15 +0800 |
|---|---|---|
| committer | Herbert Xu <herbert@gondor.apana.org.au> | 2010-05-03 11:08:15 +0800 |
| commit | a8f23ef06f619971e6a949ed7d4037820dbd4517 (patch) | |
| tree | 346df60badb9b35a31b4169425e6545d223aac19 /crypto/algapi.c | |
| parent | 5a803506702651e44c87e1f7541db84ff826f610 (diff) | |
| download | linux-crypto-a8f23ef06f619971e6a949ed7d4037820dbd4517.tar.gz linux-crypto-a8f23ef06f619971e6a949ed7d4037820dbd4517.zip | |
crypto: algapi - Remove unneeded null check
We don't check "frontend" consistently in crypto_init_spawn2(). We
check it at the start of the function but then we dereference it
unconditionally in the parameter list when we call crypto_init_spawn().
I looked at the places that call crypto_init_spawn2() and "frontend" is
always a valid pointer so I removed the check for null.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to '')
| -rw-r--r-- | crypto/algapi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/algapi.c b/crypto/algapi.c index 3e4524e6..d49d7091 100644 --- a/crypto/algapi.c +++ b/crypto/algapi.c @@ -543,7 +543,7 @@ int crypto_init_spawn2(struct crypto_spawn *spawn, struct crypto_alg *alg, { int err = -EINVAL; - if (frontend && (alg->cra_flags ^ frontend->type) & frontend->maskset) + if ((alg->cra_flags ^ frontend->type) & frontend->maskset) goto out; spawn->frontend = frontend; |