linux-stable.git/fs/namei.c, branch linux-3.3.y

vfs: fix return value from do_last()

2012-03-10T22:05:30+00:00

complete_walk() returns either ECHILD or ESTALE.  do_last() turns this into
ECHILD unconditionally.  If not in RCU mode, this error will reach userspace
which is complete nonsense.

Signed-off-by: Miklos Szeredi 
CC: stable@vger.kernel.org
Signed-off-by: Al Viro

vfs: fix double put after complete_walk()

2012-03-10T22:05:30+00:00

complete_walk() already puts nd->path, no need to do it again at cleanup time.

This would result in Oopses if triggered, apparently the codepath is not too
well exercised.

Signed-off-by: Miklos Szeredi 
CC: stable@vger.kernel.org
Signed-off-by: Al Viro

vfs: export full_name_hash() function to modules

2012-03-03T03:40:57+00:00

Commit 5707c87f "vfs: uninline full_name_hash()" broke the modular
build, because it needs exporting now that it isn't inlined any more.

Reported-by: Tetsuo Handa 
Signed-off-by: Linus Torvalds

vfs: split up name hashing in link_path_walk() into helper function

2012-03-02T22:49:24+00:00

The code in link_path_walk() that finds out the length and the hash of
the next path component is some of the hottest code in the kernel.  And
I have a version of it that does things at the full width of the CPU
wordsize at a time, but that means that we *really* want to split it up
into a separate helper function.

So this re-organizes the code a bit and splits the hashing part into a
helper function called "hash_name()".  It returns the length of the
pathname component, while at the same time computing and writing the
hash to the appropriate location.

The code generation is slightly changed by this patch, but generally for
the better - and the added abstraction actually makes the code easier to
read too.  And the new interface is well suited for replacing just the
"hash_name()" function with alternative implementations.

Signed-off-by: Linus Torvalds

vfs: uninline full_name_hash()

2012-03-02T22:32:59+00:00

.. and also use it in lookup_one_len() rather than open-coding it.

There aren't any performance-critical users, so inlining it is silly.
But it wouldn't matter if it wasn't for the fact that the word-at-a-time
dentry name patches want to conditionally replace the function, and
uninlining it sets the stage for that.

So again, this is a preparatory patch that doesn't change any semantics,
and only prepares for a much cleaner and testable word-at-a-time dentry
name accessor patch.

Signed-off-by: Linus Torvalds

vfs: fix d_inode_lookup() dentry ref leak

2012-02-14T01:45:37+00:00

d_inode_lookup() leaks a dentry reference on IS_DEADDIR().

Signed-off-by: Miklos Szeredi 
CC: stable@vger.kernel.org
Signed-off-by: Al Viro

audit: do not call audit_getname on error

2012-01-17T21:17:01+00:00

Just a code cleanup really.  We don't need to make a function call just for
it to return on error.  This also makes the VFS function even easier to follow
and removes a conditional on a hot path.

Signed-off-by: Eric Paris

Merge branches 'vfsmount-guts', 'umode_t' and 'partitions' into Z

2012-01-07T04:15:54+00:00

vfs: move mnt_mountpoint to struct mount

2012-01-04T03:57:05+00:00

Signed-off-by: Al Viro

vfs: now it can be done - make mnt_parent point to struct mount

2012-01-04T03:57:05+00:00

Signed-off-by: Al Viro