ovl: Fix opaque regression in ovl_lookup
authorhujianyang <hujianyang@huawei.com>
Tue, 6 Jan 2015 08:10:01 +0000 (16:10 +0800)
committerMiklos Szeredi <mszeredi@suse.cz>
Thu, 8 Jan 2015 13:47:20 +0000 (14:47 +0100)
commita425c037f3dd8a56469158ab5f37beb46402d958
treeb412f83087392ee4b96c6be8437c782ad53a775b
parent2f83fd8c2849a388082f30d755a75c1e67c4643b
ovl: Fix opaque regression in ovl_lookup

Current multi-layer support overlayfs has a regression in
.lookup(). If there is a directory in upperdir and a regular
file has same name in lowerdir in a merged directory, lower
file is hidden and upper directory is set to opaque in former
case. But it is changed in present code.

In lowerdir lookup path, if a found inode is not directory,
the type checking of previous inode is missing. This inode
will be copied to the lowerstack of ovl_entry directly.

That will lead to several wrong conditions, for example,
the reading of the directory in upperdir may return an error
like:

   ls: reading directory .: Not a directory

This patch makes the lowerdir lookup path check the opaque
for non-directory file too.

Signed-off-by: hujianyang <hujianyang@huawei.com>
Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>
fs/overlayfs/super.c