Jeff Mahoney 133b8f9d63 ecryptfs: fix use with tmpfs by removing d_drop from ecryptfs_destroy_inode ... Since tmpfs has no persistent storage, it pins all its dentries in memory so they have d_count=1 when other file systems would have d_count=0. ->lookup is only used to create new dentries. If the caller doesn't instantiate it, it's freed immediately at dput(). ->readdir reads directly from the dcache and depends on the dentries being hashed. When an ecryptfs mount is mounted, it associates the lower file and dentry with the ecryptfs files as they're accessed. When it's umounted and destroys all the in-memory ecryptfs inodes, it fput's the lower_files and d_drop's the lower_dentries. Commit 4981e081 added this and a d_delete in 2008 and several months later commit caeeeecf removed the d_delete. I believe the d_drop() needs to be removed as well. The d_drop effectively hides any file that has been accessed via ecryptfs from the underlying tmpfs since it depends on it being hashed for it to be accessible. I've removed the d_drop on my development node and see no ill effects with basic testing on both tmpfs and persistent storage. As a side effect, after ecryptfs d_drops the dentries on tmpfs, tmpfs BUGs on umount. This is due to the dentries being unhashed. tmpfs->kill_sb is kill_litter_super which calls d_genocide to drop the reference pinning the dentry. It skips unhashed and negative dentries, but shrink_dcache_for_umount_subtree doesn't. Since those dentries still have an elevated d_count, we get a BUG(). This patch removes the d_drop call and fixes both issues. This issue was reported at: https://bugzilla.novell.com/show_bug.cgi?id=567887 Reported-by: Árpád Bíró <biroa@demasz.hu> Signed-off-by: Jeff Mahoney <jeffm@suse.com> Cc: Dustin Kirkland <kirkland@canonical.com> Cc: stable@kernel.org Signed-off-by: Tyler Hicks <tyhicks@linux.vnet.ibm.com>		2010-04-19 14:42:13 -05:00
..
crypto.c	eCryptfs: Strip metadata in xattr flag in encrypted view	2010-03-23 12:31:35 -05:00
debug.c	eCryptfs: update comment and debug statement	2007-10-16 09:43:11 -07:00
dentry.c	fsstack/ecryptfs: remove unused get_nlinks param to fsstack_copy_attr_all	2009-12-17 10:57:30 -05:00
ecryptfs_kernel.h	eCryptfs: Strip metadata in xattr flag in encrypted view	2010-03-23 12:31:35 -05:00
file.c	ecryptfs: Eliminate useless code	2010-01-19 22:36:05 -06:00
inode.c	ecryptfs: fix error code for missing xattrs in lower fs	2010-04-19 14:42:09 -05:00
Kconfig	eCryptfs: Remove Kconfig NET dependency and select MD5	2009-10-08 11:31:36 -05:00
keystore.c	eCryptfs: Validate global auth tok keys	2009-09-23 09:10:32 -05:00
kthread.c	eCryptfs: Check for O_RDONLY lower inodes when opening lower files	2009-09-23 09:10:32 -05:00
main.c	ecryptfs: fix interpose/interpolate typos in comments	2010-01-19 22:36:03 -06:00
Makefile	eCryptfs: remove netlink transport	2008-10-16 11:21:39 -07:00
messaging.c	eCryptfs: NULL pointer dereference in ecryptfs_send_miscdev()	2009-04-22 03:54:13 -05:00
miscdev.c	eCryptfs: NULL pointer dereference in ecryptfs_send_miscdev()	2009-04-22 03:54:13 -05:00
mmap.c	eCryptfs: Strip metadata in xattr flag in encrypted view	2010-03-23 12:31:35 -05:00
read_write.c	eCryptfs: Propagate vfs_read and vfs_write return codes	2009-09-23 09:10:34 -05:00
super.c	ecryptfs: fix use with tmpfs by removing d_drop from ecryptfs_destroy_inode	2010-04-19 14:42:13 -05:00