linux.git/fs/hostfs, branch v2.6.36

hostfs ->follow_link() braino

2010-08-18T10:21:10+00:00

we want the assignment to err done inside the if () to be
visible after it, so (re)declaring err inside if () body
is wrong.

Signed-off-by: Al Viro

hostfs: dumb (and usually harmless) tpyo - strncpy instead of strlcpy

2010-08-18T10:18:57+00:00

... not harmless in this case - we have a string in the end of buffer
already.

Signed-off-by: Al Viro

try to get rid of races in hostfs open()

2010-08-09T20:48:15+00:00

In case of mode mismatch, do *not* blindly close the descriptor
another openers might be using right now.  Open the underlying
file with currently sufficient mode, then
	* if current mode has grown so that it's sufficient for
us now, just close our new fd
	* if current mode has grown and our fd is *not* enough
to cover it, close and repeat.
	* otherwise, install our fd if the file hadn't been
opened at all or dup2() our fd over the current one (and close
our fd).
Critical section is protected by mutex; yes, system-wide.  All
we do under it is a bunch of comparison and maybe an overwriting
dup2() on host.

Signed-off-by: Al Viro

leak in hostfs_unlink()

2010-08-09T20:48:14+00:00

Signed-off-by: Al Viro

hostfs: fix races in dentry_name() and inode_name()

2010-08-09T20:48:14+00:00

calculating size, then doing allocation, then filling the
path is a Bad Idea(tm), since the ancestors can be renamed,
leading to buffer overrun.

Signed-off-by: Al Viro

hostfs: sanitize symlinks

2010-08-09T20:48:12+00:00

Signed-off-by: Al Viro

hostfs: get rid of inode_dentry_name()

2010-08-09T20:48:11+00:00

it's equivalent to dentry_name() anyway

Signed-off-by: Al Viro

hostfs: get rid of file_type(), fold init_inode()

2010-08-09T20:48:10+00:00

Signed-off-by: Al Viro

switch stat_file() to passing a single struct rather than fsckloads of pointers

2010-08-09T20:48:10+00:00

Signed-off-by: Al Viro

hostfs: pass pathname to init_inode()

2010-08-09T20:48:09+00:00

We will calculate it in all callers anyway, so there's no
need to duplicate that inside.  Moreover, that way we lose
all failure exits in init_inode(), so it doesn't need to
return anything.

Signed-off-by: Al Viro