mempool_destroy() already checks for a NULL value being passed in, this 
eliminates duplicate checks.

This was caught by running make coccicheck M=block/ on linus' tree on commit 
77ede3a014a32746002f7889211f0cecf4803163 (current head as of this patch).

Signed-off-by: Tim Hansen <devtimhan...@gmail.com>
---
 block/bio-integrity.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/block/bio-integrity.c b/block/bio-integrity.c
index 5df3290..23b42e8 100644
--- a/block/bio-integrity.c
+++ b/block/bio-integrity.c
@@ -485,11 +485,8 @@ EXPORT_SYMBOL(bioset_integrity_create);
 
 void bioset_integrity_free(struct bio_set *bs)
 {
-       if (bs->bio_integrity_pool)
-               mempool_destroy(bs->bio_integrity_pool);
-
-       if (bs->bvec_integrity_pool)
-               mempool_destroy(bs->bvec_integrity_pool);
+       mempool_destroy(bs->bio_integrity_pool);
+       mempool_destroy(bs->bvec_integrity_pool);
 }
 EXPORT_SYMBOL(bioset_integrity_free);
 
-- 
2.1.4

Reply via email to