diff options
author | Tim Hansen <devtimhansen@gmail.com> | 2017-10-05 14:09:20 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2017-10-06 15:03:12 -0400 |
commit | 4b14a5c5d57f4fd6929db3427ba4d7c3775b4680 (patch) | |
tree | f4897492ababa6c43169489e3fc10985983efa87 /block/bio-integrity.c | |
parent | 775d3a35dc3e13de55ec0e061c59e36faa7dd7f0 (diff) |
block: remove unnecessary NULL checks in bioset_integrity_free()
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).
Reviewed-by: Kyle Fortin <kyle.fortin@oracle.com>
Acked-by: Martin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: Tim Hansen <devtimhansen@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/bio-integrity.c')
-rw-r--r-- | block/bio-integrity.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/block/bio-integrity.c b/block/bio-integrity.c index 5df32907ff3b..23b42e8aa03e 100644 --- a/block/bio-integrity.c +++ b/block/bio-integrity.c | |||
@@ -485,11 +485,8 @@ EXPORT_SYMBOL(bioset_integrity_create); | |||
485 | 485 | ||
486 | void bioset_integrity_free(struct bio_set *bs) | 486 | void bioset_integrity_free(struct bio_set *bs) |
487 | { | 487 | { |
488 | if (bs->bio_integrity_pool) | 488 | mempool_destroy(bs->bio_integrity_pool); |
489 | mempool_destroy(bs->bio_integrity_pool); | 489 | mempool_destroy(bs->bvec_integrity_pool); |
490 | |||
491 | if (bs->bvec_integrity_pool) | ||
492 | mempool_destroy(bs->bvec_integrity_pool); | ||
493 | } | 490 | } |
494 | EXPORT_SYMBOL(bioset_integrity_free); | 491 | EXPORT_SYMBOL(bioset_integrity_free); |
495 | 492 | ||