]> git.baikalelectronics.ru Git - kernel.git/commit
Btrfs: fix use-after-free during inode eviction
authorFilipe Manana <fdmanana@suse.com>
Fri, 12 Oct 2018 12:02:48 +0000 (13:02 +0100)
committerDavid Sterba <dsterba@suse.com>
Fri, 19 Oct 2018 10:25:33 +0000 (12:25 +0200)
commit4cbd02da82c2d749502f3bd08347427d23ae6dfd
treea4c64d9898393ee6aa83016fd4d0c5be423b9a02
parent5961aaf2a3c00ef00b0c03caad79f7bf862f099f
Btrfs: fix use-after-free during inode eviction

At inode.c:evict_inode_truncate_pages(), when we iterate over the
inode's extent states, we access an extent state record's "state" field
after we unlocked the inode's io tree lock. This can lead to a
use-after-free issue because after we unlock the io tree that extent
state record might have been freed due to being merged into another
adjacent extent state record (a previous inflight bio for a read
operation finished in the meanwhile which unlocked a range in the io
tree and cause a merge of extent state records, as explained in the
comment before the while loop added in commit 57753ffa1c6a7 ("Btrfs: fix
hang during inode eviction due to concurrent readahead")).

Fix this by keeping a copy of the extent state's flags in a local
variable and using it after unlocking the io tree.

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=201189
Fixes: a512ce4c131b ("btrfs: Add handler for invalidate page")
CC: stable@vger.kernel.org # 4.4+
Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/inode.c