Btrfs: fix crash of btrfs_release_extent_buffer_page
authorLiu Bo <bo.li.liu@oracle.com>
Tue, 23 Sep 2014 14:22:33 +0000 (22:22 +0800)
committerChris Mason <clm@fb.com>
Fri, 3 Oct 2014 23:14:58 +0000 (16:14 -0700)
commit8146502820231da924ca22d147cdcf733ac9a057
treea4ff285ca64394aea238a249ac9bf41fbc7fbe11
parent55e3bd2e0c2e1cfb43429b962e61415e0526bc01
Btrfs: fix crash of btrfs_release_extent_buffer_page

This is actually inspired by Filipe's patch.  When write_one_eb() fails on
submit_extent_page(), it'll give up writing this eb and mark it with
EXTENT_BUFFER_IOERR.  So if it's not the last page that encounter the failure,
there are some left pages which remain DIRTY, and if a later COW on this eb
happens, ie. eb is COWed and freed, it'd run into BUG_ON in
btrfs_release_extent_buffer_page() for the DIRTY page, ie. BUG_ON(PageDirty(page));

This adds the missing clear_page_dirty_for_io() for the rest pages of eb.

Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
Reviewed-by: Filipe Manana <fdmanana@suse.com>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/extent_io.c