diff options
| author | Dan Carpenter | 2014-11-22 19:36:28 +0100 |
|---|---|---|
| committer | Herbert Xu | 2014-11-25 15:50:43 +0100 |
| commit | 5d1b3c98ec2015036349c57289711f26c2d5fe01 (patch) | |
| tree | f478eb063f2b7e7a1baea4a7ccdef3754afc9be7 /arch/x86/crypto | |
| parent | crytpo: qat - Fix 64 bytes requests (diff) | |
| download | kernel-qcow2-linux-5d1b3c98ec2015036349c57289711f26c2d5fe01.tar.gz kernel-qcow2-linux-5d1b3c98ec2015036349c57289711f26c2d5fe01.tar.xz kernel-qcow2-linux-5d1b3c98ec2015036349c57289711f26c2d5fe01.zip | |
crypto: sha-mb - remove a bogus NULL check
This can't be NULL and we dereferenced it earlier. Smatch used to
ignore these things where the pointer was obviously non-NULL but I've
found that sometimes the intention was to check something else so we
were maybe missing bugs.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Tim Chen <tim.c.chen@linux.intel.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Diffstat (limited to 'arch/x86/crypto')
| -rw-r--r-- | arch/x86/crypto/sha-mb/sha1_mb.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/x86/crypto/sha-mb/sha1_mb.c b/arch/x86/crypto/sha-mb/sha1_mb.c index 99eefd812958..a225a5ca1037 100644 --- a/arch/x86/crypto/sha-mb/sha1_mb.c +++ b/arch/x86/crypto/sha-mb/sha1_mb.c @@ -204,8 +204,7 @@ static struct sha1_hash_ctx *sha1_ctx_mgr_resubmit(struct sha1_ctx_mgr *mgr, str continue; } - if (ctx) - ctx->status = HASH_CTX_STS_IDLE; + ctx->status = HASH_CTX_STS_IDLE; return ctx; } |