summaryrefslogtreecommitdiffstats
path: root/fs/cramfs
diff options
context:
space:
mode:
authorJeff Layton <jlayton@redhat.com>2008-07-12 13:47:59 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2008-07-12 14:33:42 -0700
commite911d0cc877ff027d5bd09fc33148ab76f0fdf0e (patch)
treec2549fb1caa266f17d25c20e0c499db823578372 /fs/cramfs
parentd3297a644a0ab784e0c810ceca6bf35a67868ad9 (diff)
downloadlinux-e911d0cc877ff027d5bd09fc33148ab76f0fdf0e.tar.gz
linux-e911d0cc877ff027d5bd09fc33148ab76f0fdf0e.tar.bz2
linux-e911d0cc877ff027d5bd09fc33148ab76f0fdf0e.zip
cifs: fix inode leak in cifs_get_inode_info_unix
Try this: mount a share with unix extensions create a file on it umount the share You'll get the following message in the ring buffer: VFS: Busy inodes after unmount of cifs. Self-destruct in 5 seconds. Have a nice day... ...the problem is that cifs_get_inode_info_unix is creating and hashing a new inode even when it's going to return error anyway. The first lookup when creating a file returns an error so we end up leaking this inode before we do the actual create. This appears to be a regression caused by commit 0e4bbde94fdc33f5b3d793166b21bf768ca3e098. The following patch seems to fix it for me, and fixes a minor formatting nit as well. Signed-off-by: Jeff Layton <jlayton@redhat.com> Acked-by: Steven French <sfrench@us.ibm.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/cramfs')
0 files changed, 0 insertions, 0 deletions