diff options
| author | Richard Weinberger <richard@nod.at> | 2010-10-26 17:22:20 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-10-26 19:52:12 -0400 |
| commit | c5c6dd4e2dce3cb4d705d97183bc42b4e33e2606 (patch) | |
| tree | 44d3d1cb85a2d44c3160bff06ce0467d984b7f99 /fs/hostfs | |
| parent | be76d81f99c4c120adcd201a7316e4dd7dbe3c11 (diff) | |
hostfs: code cleanups
Some code cleanups for hostfs.
Signed-off-by: Richard Weinberger <richard@nod.at>
Cc: Jeff Dike <jdike@addtoit.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/hostfs')
| -rw-r--r-- | fs/hostfs/hostfs_user.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/hostfs/hostfs_user.c b/fs/hostfs/hostfs_user.c index 8d02683585e0..d51a98384bc0 100644 --- a/fs/hostfs/hostfs_user.c +++ b/fs/hostfs/hostfs_user.c | |||
| @@ -94,8 +94,7 @@ void *open_dir(char *path, int *err_out) | |||
| 94 | 94 | ||
| 95 | dir = opendir(path); | 95 | dir = opendir(path); |
| 96 | *err_out = errno; | 96 | *err_out = errno; |
| 97 | if (dir == NULL) | 97 | |
| 98 | return NULL; | ||
| 99 | return dir; | 98 | return dir; |
| 100 | } | 99 | } |
| 101 | 100 | ||
| @@ -205,7 +204,7 @@ int set_attr(const char *file, struct hostfs_iattr *attrs, int fd) | |||
| 205 | if (attrs->ia_valid & HOSTFS_ATTR_MODE) { | 204 | if (attrs->ia_valid & HOSTFS_ATTR_MODE) { |
| 206 | if (fd >= 0) { | 205 | if (fd >= 0) { |
| 207 | if (fchmod(fd, attrs->ia_mode) != 0) | 206 | if (fchmod(fd, attrs->ia_mode) != 0) |
| 208 | return (-errno); | 207 | return -errno; |
| 209 | } else if (chmod(file, attrs->ia_mode) != 0) { | 208 | } else if (chmod(file, attrs->ia_mode) != 0) { |
| 210 | return -errno; | 209 | return -errno; |
| 211 | } | 210 | } |
