aboutsummaryrefslogtreecommitdiff
path: root/fs/ext4/fast_commit.c
diff options
context:
space:
mode:
authorBaokun Li <libaokun1@huawei.com>2024-10-08 20:11:52 +0800
committerTheodore Ts'o <tytso@mit.edu>2024-11-12 23:49:55 -0500
commit40eb3104cf416188c4bd8fd7946f6c388409c286 (patch)
tree77b1fafd5c5e3d2d1c087830e6bac759450148e3 /fs/ext4/fast_commit.c
parentfdfa648ab9393de8c1be21cad17c17bdced3a68a (diff)
downloadlinux-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