diff options
author | zhong jiang <zhongjiang@huawei.com> | 2018-09-21 09:30:15 -0400 |
---|---|---|
committer | Herbert Xu <herbert@gondor.apana.org.au> | 2018-09-28 00:46:27 -0400 |
commit | 79517e8f8e6dddd2a90517123ffdb366828b69f5 (patch) | |
tree | 8141f83577defc6e94a3023537720991ac7e2845 | |
parent | 2a8780be9c26457b84d786f7ac9ed110625dde92 (diff) |
crypto: cavium - remove redundant null pointer check before kfree
kfree has taken the null pointer into account. hence it is safe
to remove the redundant null pointer check before kfree.
Signed-off-by: zhong jiang <zhongjiang@huawei.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
-rw-r--r-- | drivers/crypto/cavium/cpt/cptvf_reqmanager.c | 20 |
1 files changed, 5 insertions, 15 deletions
diff --git a/drivers/crypto/cavium/cpt/cptvf_reqmanager.c b/drivers/crypto/cavium/cpt/cptvf_reqmanager.c index b0ba4331944b..ca549c5dc08e 100644 --- a/drivers/crypto/cavium/cpt/cptvf_reqmanager.c +++ b/drivers/crypto/cavium/cpt/cptvf_reqmanager.c | |||
@@ -308,21 +308,11 @@ void do_request_cleanup(struct cpt_vf *cptvf, | |||
308 | } | 308 | } |
309 | } | 309 | } |
310 | 310 | ||
311 | if (info->scatter_components) | 311 | kzfree(info->scatter_components); |
312 | kzfree(info->scatter_components); | 312 | kzfree(info->gather_components); |
313 | 313 | kzfree(info->out_buffer); | |
314 | if (info->gather_components) | 314 | kzfree(info->in_buffer); |
315 | kzfree(info->gather_components); | 315 | kzfree((void *)info->completion_addr); |
316 | |||
317 | if (info->out_buffer) | ||
318 | kzfree(info->out_buffer); | ||
319 | |||
320 | if (info->in_buffer) | ||
321 | kzfree(info->in_buffer); | ||
322 | |||
323 | if (info->completion_addr) | ||
324 | kzfree((void *)info->completion_addr); | ||
325 | |||
326 | kzfree(info); | 316 | kzfree(info); |
327 | } | 317 | } |
328 | 318 | ||