linux/fs/ecryptfs, branch v2.6.35

ecryptfs: Bugfix for error related to ecryptfs_hash_buckets

2010-07-29T02:59:24Z

The function ecryptfs_uid_hash wrongly assumes that the second parameter to hash_long() is the number of hash buckets instead of the number of hash bits. This patch fixes that and renames the variable ecryptfs_hash_buckets to ecryptfs_hash_bits to make it clearer. Fixes: CVE-2010-2492 Signed-off-by: Andre Osterhues Signed-off-by: Tyler Hicks Signed-off-by: Linus Torvalds

kill spurious reference to vmtruncate

2010-05-28T02:15:42Z

Lots of filesystems calls vmtruncate despite not implementing the old ->truncate method. Switch them to use simple_setsize and add some comments about the truncate code where it seems fitting. Signed-off-by: Christoph Hellwig Signed-off-by: Nick Piggin Signed-off-by: Al Viro

drop unused dentry argument to ->fsync

2010-05-28T02:05:02Z

Signed-off-by: Christoph Hellwig Signed-off-by: Al Viro

switch ecryptfs_write() to struct inode *, kill on-stack fake files

2010-05-21T22:31:28Z

Signed-off-by: Al Viro

switch ecryptfs_get_locked_page() to struct inode *

2010-05-21T22:31:28Z

Signed-off-by: Al Viro

simplify access to ecryptfs inodes in ->readpage() and friends

2010-05-21T22:31:28Z

we can get to them from page->mapping->host, no need to mess with file. Signed-off-by: Al Viro

Ban ecryptfs over ecryptfs

2010-05-21T22:31:27Z

This is a seriously simplified patch from Eric Sandeen; copy of rationale follows: === mounting stacked ecryptfs on ecryptfs has been shown to lead to bugs in testing. For crypto info in xattr, there is no mechanism for handling this at all, and for normal file headers, we run into other trouble: BUG: unable to handle kernel NULL pointer dereference at 0000000000000008 IP: [] ecryptfs_d_revalidate+0x43/0xa0 [ecryptfs] ... There doesn't seem to be any good usecase for this, so I'd suggest just disallowing the configuration. Based on a patch originally, I believe, from Mike Halcrow. === Signed-off-by: Al Viro

sanitize vfs_fsync calling conventions

2010-05-21T22:31:21Z

Now that the last user passing a NULL file pointer is gone we can remove the redundant dentry argument and associated hacks inside vfs_fsynmc_range. The next step will be removig the dentry argument from ->fsync, but given the luck with the last round of method prototype changes I'd rather defer this until after the main merge window. Signed-off-by: Christoph Hellwig Signed-off-by: Al Viro

Clean ecryptfs ->get_sb() up

2010-05-21T22:31:13Z

Signed-off-by: Al Viro

fix a couple of ecryptfs leaks

2010-05-21T22:31:13Z

First of all, get_sb_nodev() grabs anon dev minor and we never free it in ecryptfs ->kill_sb(). Moreover, on one of the failure exits in ecryptfs_get_sb() we leak things - it happens before we set ->s_root and ->put_super() won't be called in that case. Solution: kill ->put_super(), do all that stuff in ->kill_sb(). And use kill_anon_sb() instead of generic_shutdown_super() to deal with anon dev leak. Signed-off-by: Al Viro