diff options
author | Eric Whitney <enwlinux@gmail.com> | 2014-04-01 19:49:30 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2014-04-01 19:49:30 -0400 |
commit | ad6599ab3ac98a4474544086e048ce86ec15a4d1 (patch) | |
tree | e6684f38f28eff00aece58ebac96d4bdcd3dfa79 /fs/fat | |
parent | e5b30416f3631bf4eed37c3bec9f789f9ae78446 (diff) | |
download | linux-ad6599ab3ac98a4474544086e048ce86ec15a4d1.tar.bz2 |
ext4: fix premature freeing of partial clusters split across leaf blocks
Xfstests generic/311 and shared/298 fail when run on a bigalloc file
system. Kernel error messages produced during the tests report that
blocks to be freed are already on the to-be-freed list. When e2fsck
is run at the end of the tests, it typically reports bad i_blocks and
bad free blocks counts.
The bug that causes these failures is located in ext4_ext_rm_leaf().
Code at the end of the function frees a partial cluster if it's not
shared with an extent remaining in the leaf. However, if all the
extents in the leaf have been removed, the code dereferences an
invalid extent pointer (off the front of the leaf) when the check for
sharing is made. This generally has the effect of unconditionally
freeing the partial cluster, which leads to the observed failures
when the partial cluster is shared with the last extent in the next
leaf.
Fix this by attempting to free the cluster only if extents remain in
the leaf. Any remaining partial cluster will be freed if possible
when the next leaf is processed or when leaf removal is complete.
Signed-off-by: Eric Whitney <enwlinux@gmail.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Cc: stable@vger.kernel.org
Diffstat (limited to 'fs/fat')
0 files changed, 0 insertions, 0 deletions