Commit 144da23b authored by Lubos Dolezel's avatar Lubos Dolezel Committed by Miklos Szeredi
Browse files

ovl: return required buffer size for file handles



Overlayfs doesn't work well with the fanotify mechanism.

Fanotify first probes for the required buffer size for the file handle,
but overlayfs currently bails out without passing the size back.

That results in errors in the kernel log, such as:

[527944.485384] overlayfs: failed to encode file handle (/, err=-75, buflen=0, len=29, type=1)
[527944.485386] fanotify: failed to encode fid (fsid=ae521e68.a434d95f, type=255, bytes=0, err=-2)

Signed-off-by: default avatarLubos Dolezel <lubos@dolezel.info>
Signed-off-by: default avatarAmir Goldstein <amir73il@gmail.com>
Signed-off-by: default avatarMiklos Szeredi <mszeredi@redhat.com>
parent 399c109d
Loading
Loading
Loading
Loading
+8 −10
Original line number Diff line number Diff line
@@ -231,11 +231,8 @@ static int ovl_dentry_to_fid(struct dentry *dentry, u32 *fid, int buflen)
	if (IS_ERR(fh))
		return PTR_ERR(fh);

	err = -EOVERFLOW;
	len = OVL_FH_LEN(fh);
	if (len > buflen)
		goto fail;

	if (len <= buflen)
		memcpy(fid, fh, len);
	err = len;

@@ -244,9 +241,8 @@ out:
	return err;

fail:
	pr_warn_ratelimited("failed to encode file handle (%pd2, err=%i, buflen=%d, len=%d, type=%d)\n",
			    dentry, err, buflen, fh ? (int)fh->fb.len : 0,
			    fh ? fh->fb.type : 0);
	pr_warn_ratelimited("failed to encode file handle (%pd2, err=%i)\n",
			    dentry, err);
	goto out;
}

@@ -254,7 +250,7 @@ static int ovl_encode_fh(struct inode *inode, u32 *fid, int *max_len,
			 struct inode *parent)
{
	struct dentry *dentry;
	int bytes = *max_len << 2;
	int bytes, buflen = *max_len << 2;

	/* TODO: encode connectable file handles */
	if (parent)
@@ -264,12 +260,14 @@ static int ovl_encode_fh(struct inode *inode, u32 *fid, int *max_len,
	if (WARN_ON(!dentry))
		return FILEID_INVALID;

	bytes = ovl_dentry_to_fid(dentry, fid, bytes);
	bytes = ovl_dentry_to_fid(dentry, fid, buflen);
	dput(dentry);
	if (bytes <= 0)
		return FILEID_INVALID;

	*max_len = bytes >> 2;
	if (bytes > buflen)
		return FILEID_INVALID;

	return OVL_FILEID_V1;
}