mirror of
https://github.com/torvalds/linux.git
synced 2026-09-25 09:41:03 +02:00
ocfs2: do not use make_bad_inode() in ocfs2_read_inode_block_full()
This reverts commit58b6fcd2ab("ocfs2: mark inode bad upon validation failure during read"). Since 'make_bad_inode()' resets inode type to S_IFREG, doing this for directory inode during active VFS lookup is likely to confuse the latter, including VFS_BUG_ON_INODE() triggered in this case. Link: https://lore.kernel.org/20260709060101.207262-1-dmantipov@yandex.ru Fixes:58b6fcd2ab("ocfs2: mark inode bad upon validation failure during read") Signed-off-by: Dmitry Antipov <dmantipov@yandex.ru> Suggested-by: Al Viro <viro@zeniv.linux.org.uk> Reported-by: syzbot+d222f4b7129379c3d5bc@syzkaller.appspotmail.com Closes: https://syzkaller.appspot.com/bug?extid=d222f4b7129379c3d5bc Reviewed-by: Joseph Qi <joseph.qi@linux.alibaba.com> Cc: Mark Fasheh <mark@fasheh.com> Cc: Joel Becker <jlbec@evilplan.org> Cc: Junxiao Bi <junxiao.bi@oracle.com> Cc: Changwei Ge <gechangwei@live.cn> Cc: Jun Piao <piaojun@huawei.com> Cc: Heming Zhao <heming.zhao@suse.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
This commit is contained in:
parent
03d0a3bebc
commit
d48ace6532
|
|
@ -1952,8 +1952,6 @@ int ocfs2_read_inode_block_full(struct inode *inode, struct buffer_head **bh,
|
|||
rc = ocfs2_read_blocks(INODE_CACHE(inode), OCFS2_I(inode)->ip_blkno,
|
||||
1, &tmp, flags, ocfs2_validate_inode_block);
|
||||
|
||||
if (rc < 0)
|
||||
make_bad_inode(inode);
|
||||
/* If ocfs2_read_blocks() got us a new bh, pass it up. */
|
||||
if (!rc && !*bh)
|
||||
*bh = tmp;
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user