Commit 09e10322 authored Dec 13, 2014 by Miklos Szeredi

ovl: lookup ENAMETOOLONG on lower means ENOENT



"Suppose you have in one of the lower layers a filesystem with
->lookup()-enforced upper limit on name length.  Pretty much every local fs
has one, but... they are not all equal.  255 characters is the common upper
limit, but e.g. jffs2 stops at 254, minixfs upper limit is somewhere from
14 to 60, depending upon version, etc.  You are doing a lookup for
something that is present in upper layer, but happens to be too long for
one of the lower layers.  Too bad - ENAMETOOLONG for you..."

Reported-by: Al Viro <viro@ZenIV.linux.org.uk>
Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>

parent 3e01cee3

fs/overlayfs/super.c

+7 −1

Original line number	Diff line number	Diff line
		@@ -376,8 +376,14 @@ struct dentry ovl_lookup(struct inode dir, struct dentry *dentry,
		opaque = false;
		this = ovl_lookup_real(lowerpath.dentry, &dentry->d_name);
		err = PTR_ERR(this);
		if (IS_ERR(this))
		if (IS_ERR(this)) {
		/*
		* If it's positive, then treat ENAMETOOLONG as ENOENT.
		*/
		if (err == -ENAMETOOLONG && (upperdentry \|\| ctr))
		continue;
		goto out_put;
		}
		if (!this)
		continue;
		if (ovl_is_whiteout(this)) {

Admin message