diff options
author | Miklos Szeredi <miklos@szeredi.hu> | 2006-10-17 03:10:08 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-17 11:18:45 -0400 |
commit | 8da5ff23ce0a84d9845b01e6fe5047e17836bf5a (patch) | |
tree | dc711306a3793c91a5df7fc0223738a40ea3fb7c /fs/fuse/dir.c | |
parent | 7762f5a0b709b415fda132258ad37b9f2a1db994 (diff) |
[PATCH] fuse: locking fix for nlookup
An inode could be returned by independent parallel lookups, in this case an
update of the lookup counter could be lost resulting in a memory leak in
userspace.
Signed-off-by: Miklos Szeredi <miklos@szeredi.hu>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/fuse/dir.c')
-rw-r--r-- | fs/fuse/dir.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/fuse/dir.c b/fs/fuse/dir.c index a8f65c11aa2c..7ecfe95795cd 100644 --- a/fs/fuse/dir.c +++ b/fs/fuse/dir.c | |||
@@ -163,7 +163,9 @@ static int fuse_dentry_revalidate(struct dentry *entry, struct nameidata *nd) | |||
163 | fuse_send_forget(fc, req, outarg.nodeid, 1); | 163 | fuse_send_forget(fc, req, outarg.nodeid, 1); |
164 | return 0; | 164 | return 0; |
165 | } | 165 | } |
166 | spin_lock(&fc->lock); | ||
166 | fi->nlookup ++; | 167 | fi->nlookup ++; |
168 | spin_unlock(&fc->lock); | ||
167 | } | 169 | } |
168 | fuse_put_request(fc, req); | 170 | fuse_put_request(fc, req); |
169 | if (err || (outarg.attr.mode ^ inode->i_mode) & S_IFMT) | 171 | if (err || (outarg.attr.mode ^ inode->i_mode) & S_IFMT) |