Commit a6aa62a0 authored by Nick Piggin's avatar Nick Piggin Committed by Linus Torvalds
Browse files

mm/vmscan.c: fix mapping use after free



We need lock_page_nosync() here because we have no reference to the
mapping when taking the page lock.
Signed-off-by: default avatarNick Piggin <npiggin@suse.de>
Reviewed-by: default avatarJohannes Weiner <hannes@cmpxchg.org>
Cc: Mel Gorman <mel@csn.ul.ie>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent c61284e9
......@@ -298,7 +298,7 @@ static int may_write_to_queue(struct backing_dev_info *bdi)
static void handle_write_error(struct address_space *mapping,
struct page *page, int error)
{
lock_page(page);
lock_page_nosync(page);
if (page_mapping(page) == mapping)
mapping_set_error(mapping, error);
unlock_page(page);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment