summaryrefslogtreecommitdiffstats
path: root/fs/fuse/dir.c
diff options
context:
space:
mode:
authorMiklos Szeredi <mszeredi@suse.cz>2013-02-04 15:57:42 +0100
committerMiklos Szeredi <mszeredi@suse.cz>2013-02-04 15:57:42 +0100
commitdfca7cebc2679f3d129f8e680a8f199a7ad16e38 (patch)
treefa519439bec5f96132af4772618f53a30d0a179f /fs/fuse/dir.c
parent6a4e922c3db06f7da27e072729e047185c2fba66 (diff)
downloadop-kernel-dev-dfca7cebc2679f3d129f8e680a8f199a7ad16e38.zip
op-kernel-dev-dfca7cebc2679f3d129f8e680a8f199a7ad16e38.tar.gz
fuse: don't WARN when nlink is zero
drop_nlink() warns if nlink is already zero. This is triggerable by a buggy userspace filesystem. The cure, I think, is worse than the disease so disable the warning. Reported-by: Tero Roponen <tero.roponen@gmail.com> Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>
Diffstat (limited to 'fs/fuse/dir.c')
-rw-r--r--fs/fuse/dir.c9
1 files changed, 8 insertions, 1 deletions
diff --git a/fs/fuse/dir.c b/fs/fuse/dir.c
index dc5e648..2b112d9 100644
--- a/fs/fuse/dir.c
+++ b/fs/fuse/dir.c
@@ -705,7 +705,14 @@ static int fuse_unlink(struct inode *dir, struct dentry *entry)
spin_lock(&fc->lock);
fi->attr_version = ++fc->attr_version;
- drop_nlink(inode);
+ /*
+ * If i_nlink == 0 then unlink doesn't make sense, yet this can
+ * happen if userspace filesystem is careless. It would be
+ * difficult to enforce correct nlink usage so just ignore this
+ * condition here
+ */
+ if (inode->i_nlink > 0)
+ drop_nlink(inode);
spin_unlock(&fc->lock);
fuse_invalidate_attr(inode);
fuse_invalidate_attr(dir);
OpenPOWER on IntegriCloud