diff options
| author | Jesper Juhl <juhl-lkml@dif.dk> | 2005-07-06 13:51:00 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2005-07-06 13:51:00 -0700 |
| commit | 8279dd748f9704b811e528b31304e2fab026abc5 (patch) | |
| tree | d824994f48492dc3910ec3bf82d147cd8f796f20 /crypto | |
| parent | b4634484815e1879512a23e4f59eef648135c30a (diff) | |
| download | lwn-8279dd748f9704b811e528b31304e2fab026abc5.tar.gz lwn-8279dd748f9704b811e528b31304e2fab026abc5.zip | |
[CRYPTO] Don't check for NULL before kfree()
Checking a pointer for NULL before calling kfree() on it is redundant.
This patch removes such checks from crypto/
Signed-off-by: Jesper Juhl <juhl-lkml@dif.dk>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'crypto')
| -rw-r--r-- | crypto/cipher.c | 3 | ||||
| -rw-r--r-- | crypto/hmac.c | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/crypto/cipher.c b/crypto/cipher.c index f434ce7c2d0b..69264497b48c 100644 --- a/crypto/cipher.c +++ b/crypto/cipher.c @@ -336,6 +336,5 @@ out: void crypto_exit_cipher_ops(struct crypto_tfm *tfm) { - if (tfm->crt_cipher.cit_iv) - kfree(tfm->crt_cipher.cit_iv); + kfree(tfm->crt_cipher.cit_iv); } diff --git a/crypto/hmac.c b/crypto/hmac.c index 847df9263e16..da0456b37109 100644 --- a/crypto/hmac.c +++ b/crypto/hmac.c @@ -49,8 +49,7 @@ int crypto_alloc_hmac_block(struct crypto_tfm *tfm) void crypto_free_hmac_block(struct crypto_tfm *tfm) { - if (tfm->crt_digest.dit_hmac_block) - kfree(tfm->crt_digest.dit_hmac_block); + kfree(tfm->crt_digest.dit_hmac_block); } void crypto_hmac_init(struct crypto_tfm *tfm, u8 *key, unsigned int *keylen) |