]> git.baikalelectronics.ru Git - kernel.git/commit
Btrfs: check prepare_uptodate_page() error code earlier
authorChris Mason <clm@fb.com>
Mon, 14 Dec 2015 23:40:44 +0000 (15:40 -0800)
committerChris Mason <clm@fb.com>
Tue, 15 Dec 2015 17:09:38 +0000 (09:09 -0800)
commit9c3b81e6c76985c218fe0ef6769b35c5bf9e46d7
tree25dcd0b7dc78562d01c7865527d3fd46d6f99bb6
parent32931adca2edf07e63c3a8691a05e5b36b7e59d0
Btrfs: check prepare_uptodate_page() error code earlier

prepare_pages() may end up calling prepare_uptodate_page() twice if our
write only spans a single page.  But if the first call returns an error,
our page will be unlocked and its not safe to call it again.

This bug goes all the way back to 2011, and it's not something commonly
hit.

While we're here, add a more explicit check for the page being truncated
away.  The bare lock_page() alone is protected only by good thoughts and
i_mutex, which we're sure to regret eventually.

Reported-by: Dave Jones <dsj@fb.com>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/file.c