Whamcloud - gitweb
e2fsck: fix handling of a invalid symlink in an inline_data directory
authorTheodore Ts'o <tytso@mit.edu>
Wed, 14 Jun 2023 18:44:19 +0000 (14:44 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Wed, 14 Jun 2023 19:15:15 +0000 (15:15 -0400)
commit8798bbb81687103b0c0f56a42b096884c6032101
tree17121e6d884869cb5ab91f2f65f850ac83dbefcf
parent74571d9430da149fa46a0f7ef171920ebe9a7b8c
e2fsck: fix handling of a invalid symlink in an inline_data directory

If there is an inline directory that contains a directory entry to an
invalid symlink, and that invalid symlink is the portion of the inline
directory stored in an xattr portion of the inode, this can result in
a buffer overrun.

When check_dir_block() is handling the in-xattr portion of the inline
directory, it sets the buf pointer to the beginning of that part of
the inline directory.  This results in the scratch buffer passed to
e2fsck_process_bad_inode() to incorrect, resulting in a buffer overrun
if e2fsck_pass1_check_symlink() needs to read the symlink target (when
the symlink is too long to fit in the i_blocks[] space).

This commit fixes this by using the original cd->buf instead of buf,
since it can get modified when handling inline directories.

Fixes: 0ac4b3973f31 ("e2fsck: inspect inline dir data as two directory blocks")
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
e2fsck/pass2.c