diff options
author | Baokun Li <libaokun1@huawei.com> | 2024-10-08 20:11:52 +0800 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2024-11-12 23:49:55 -0500 |
commit | 40eb3104cf416188c4bd8fd7946f6c388409c286 (patch) | |
tree | 77b1fafd5c5e3d2d1c087830e6bac759450148e3 /fs/ext4/fast_commit.c | |
parent | fdfa648ab9393de8c1be21cad17c17bdced3a68a (diff) | |
download | linux-40eb3104cf416188c4bd8fd7946f6c388409c286.tar.gz linux-40eb3104cf416188c4bd8fd7946f6c388409c286.tar.bz2 linux-40eb3104cf416188c4bd8fd7946f6c388409c286.zip |
ext4: WARN if a full dir leaf block has only one dentry
The maximum length of a filename is 255 and the minimum block size is 1024,
so it is always guaranteed that the number of entries is greater than or
equal to 2 when do_split() is called. So unless ext4_dx_add_entry() and
make_indexed_dir() or some other functions are buggy, 'split == 0' will
not occur.
Setting 'continued' to 0 in this case masks the problem that the file
system has become corrupted, even though it prevents possible out-of-bounds
access. Hence WARN_ON_ONCE() is used to check if 'split' is 0, and if it is
then warns and returns an error to abort split.
Suggested-by: Theodore Ts'o <tytso@mit.edu>
Link: https://lore.kernel.org/r/20240823160518.GA424729@mit.edu
Signed-off-by: Baokun Li <libaokun1@huawei.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Link: https://patch.msgid.link/20241008121152.3771906-1-libaokun@huaweicloud.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
Diffstat (limited to 'fs/ext4/fast_commit.c')
0 files changed, 0 insertions, 0 deletions