Commit a3761c3c authored Apr 10, 2019 by Jérôme Glisse Committed by Jens Axboe Apr 10, 2019

block: do not leak memory in bio_copy_user_iov()

When bio_add_pc_page() fails in bio_copy_user_iov() we should free
the page we just allocated otherwise we are leaking it.

Cc: linux-block@vger.kernel.org
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: stable@vger.kernel.org
Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>
Signed-off-by: Jérôme Glisse <jglisse@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

parent b2b3a70c

block/bio.c

+4 −1

Original line number	Diff line number	Diff line
		@@ -1298,8 +1298,11 @@ struct bio bio_copy_user_iov(struct request_queue q,
		}
		}

		if (bio_add_pc_page(q, bio, page, bytes, offset) < bytes)
		if (bio_add_pc_page(q, bio, page, bytes, offset) < bytes) {
		if (!map_data)
		__free_page(page);
		break;
		}

		len -= bytes;
		offset = 0;

Admin message