diff options
| author | Justin Tee <justin.tee@broadcom.com> | 2019-12-05 10:09:01 +0800 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2019-12-05 11:38:36 -0700 |
| commit | ece841abbed2da71fa10710c687c9ce9efb6bf69 (patch) | |
| tree | 76f3d630387629bd6dcfa77163aa3aa94503cc07 /block/bio.c | |
| parent | 08802ed665e47243393f43501bdafa032112eb1c (diff) | |
| download | lwn-ece841abbed2da71fa10710c687c9ce9efb6bf69.tar.gz lwn-ece841abbed2da71fa10710c687c9ce9efb6bf69.zip | |
block: fix memleak of bio integrity data
7c20f11680a4 ("bio-integrity: stop abusing bi_end_io") moves
bio_integrity_free from bio_uninit() to bio_integrity_verify_fn()
and bio_endio(). This way looks wrong because bio may be freed
without calling bio_endio(), for example, blk_rq_unprep_clone() is
called from dm_mq_queue_rq() when the underlying queue of dm-mpath
is busy.
So memory leak of bio integrity data is caused by commit 7c20f11680a4.
Fixes this issue by re-adding bio_integrity_free() to bio_uninit().
Fixes: 7c20f11680a4 ("bio-integrity: stop abusing bi_end_io")
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by Justin Tee <justin.tee@broadcom.com>
Add commit log, and simplify/fix the original patch wroten by Justin.
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block/bio.c')
| -rw-r--r-- | block/bio.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/block/bio.c b/block/bio.c index b1170ec18464..9d54aa37ce6c 100644 --- a/block/bio.c +++ b/block/bio.c @@ -233,6 +233,9 @@ fallback: void bio_uninit(struct bio *bio) { bio_disassociate_blkg(bio); + + if (bio_integrity(bio)) + bio_integrity_free(bio); } EXPORT_SYMBOL(bio_uninit); |