]> git.ipfire.org Git - people/ms/linux.git/commit
ext4: don't orphan or truncate the boot loader inode
authorTheodore Ts'o <tytso@mit.edu>
Mon, 6 Oct 2014 02:47:07 +0000 (22:47 -0400)
committerZefan Li <lizefan@huawei.com>
Mon, 2 Feb 2015 09:04:52 +0000 (17:04 +0800)
commit07048f9e15e456be7d216e6f4515c33cafc6fc2b
tree4e6ad2fad0e9c6c43cdd4cdab6bc0d2571986c55
parent60e7100a311b7b0d4ad87f20d6a13f1f4a4d786d
ext4: don't orphan or truncate the boot loader inode

commit e2bfb088fac03c0f621886a04cffc7faa2b49b1d upstream.

The boot loader inode (inode #5) should never be visible in the
directory hierarchy, but it's possible if the file system is corrupted
that there will be a directory entry that points at inode #5.  In
order to avoid accidentally trashing it, when such a directory inode
is opened, the inode will be marked as a bad inode, so that it's not
possible to modify (or read) the inode from userspace.

Unfortunately, when we unlink this (invalid/illegal) directory entry,
we will put the bad inode on the ophan list, and then when try to
unlink the directory, we don't actually remove the bad inode from the
orphan list before freeing in-memory inode structure.  This means the
in-memory orphan list is corrupted, leading to a kernel oops.

In addition, avoid truncating a bad inode in ext4_destroy_inode(),
since truncating the boot loader inode is not a smart thing to do.

Reported-by: Sami Liedes <sami.liedes@iki.fi>
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
[lizf: Backported to 3.4: adjust context]
Signed-off-by: Zefan Li <lizefan@huawei.com>
fs/ext4/inode.c
fs/ext4/namei.c