summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorJan Kara <jack@suse.cz>2014-09-04 11:47:51 +0200
committerJan Kara <jack@suse.cz>2014-09-04 11:47:51 +0200
commit8a70ee3307908c46f952df91be72a18d5f5ad0a3 (patch)
treebf5e1b58e99f4b3090d3811aec3411a728df5f40 /fs
parent44bf091f508913c1c35e70ea96430454c95c78f1 (diff)
downloadop-kernel-dev-8a70ee3307908c46f952df91be72a18d5f5ad0a3.zip
op-kernel-dev-8a70ee3307908c46f952df91be72a18d5f5ad0a3.tar.gz
udf: Avoid dir link count to go negative
If we are writing back inode of unlinked directory, its link count ends up being (u16)-1. Although the inode is deleted, udf_iget() can load the inode when NFS uses stale file handle and get confused. Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs')
-rw-r--r--fs/udf/inode.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/udf/inode.c b/fs/udf/inode.c
index 236cd48..e86f9b6 100644
--- a/fs/udf/inode.c
+++ b/fs/udf/inode.c
@@ -1664,7 +1664,7 @@ static int udf_update_inode(struct inode *inode, int do_sync)
FE_PERM_U_DELETE | FE_PERM_U_CHATTR));
fe->permissions = cpu_to_le32(udfperms);
- if (S_ISDIR(inode->i_mode))
+ if (S_ISDIR(inode->i_mode) && inode->i_nlink > 0)
fe->fileLinkCount = cpu_to_le16(inode->i_nlink - 1);
else
fe->fileLinkCount = cpu_to_le16(inode->i_nlink);
OpenPOWER on IntegriCloud