Greg KH wrote:
-stable review patch. If anyone has any objections, please let us know.
------------------
From: Neil Brown <[email protected]>
The inode number out of an NFS file handle gets passed eventually to
ext3_get_inode_block() without any checking. If ext3_get_inode_block()
allows it to trigger an error, then bad filehandles can have unpleasant
effect - ext3_error() will usually cause a forced read-only remount, or a
panic if `errors=panic' was used.
So remove the call to ext3_error there and put a matching check in
ext3/namei.c where inode numbers are read off storage.
This patch and the ext2 patch (23/23) are accomplishing the same thing in 2
different ways, I think, and introducing unnecessary differences between ext2
and ext3. I'd personally prefer to see both ext2 and ext3 handled with the
get_dentry op addition, and I'd be happy to quickly whip up the ext3 patch to do
this if there's agreement on this path.
(there's nothing technically wrong with either approach, it just seems
inconsistent to me).
Thanks,
-Eric
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
