diff options
author | Eric Sesterhenn <snakebyte@gmx.de> | 2006-06-25 08:48:57 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-06-25 13:01:20 -0400 |
commit | 099a71d99578a53bdf5f383c55e4095f1c59410c (patch) | |
tree | e43761b2e62c5ce2591113f65596389074c92105 | |
parent | 18cddac3d17781d9cbc109522d70ab021c846e2b (diff) |
[PATCH] Remove needless checks in fs/9p/vfs_inode.c
coverity found two needless checks in vfs_inode.c (cid #1165 and #1164)
In both cases inode is always NULL when we goto error; either because it
is still initialized to NULL or is set to NULL explicitly. This patch
simply removes these checks to save some code.
Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Acked-by: Eric Van Hensbergen <ericvh@gmail.com>
Cc: Ron Minnich <rminnich@lanl.gov>
Cc: Latchesar Ionkov <lucho@ionkov.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | fs/9p/vfs_inode.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c index 8e60dc7ec4a6..5c6bdf82146c 100644 --- a/fs/9p/vfs_inode.c +++ b/fs/9p/vfs_inode.c | |||
@@ -530,9 +530,6 @@ error: | |||
530 | if (vfid) | 530 | if (vfid) |
531 | v9fs_fid_destroy(vfid); | 531 | v9fs_fid_destroy(vfid); |
532 | 532 | ||
533 | if (inode) | ||
534 | iput(inode); | ||
535 | |||
536 | return err; | 533 | return err; |
537 | } | 534 | } |
538 | 535 | ||
@@ -1174,9 +1171,6 @@ error: | |||
1174 | if (vfid) | 1171 | if (vfid) |
1175 | v9fs_fid_destroy(vfid); | 1172 | v9fs_fid_destroy(vfid); |
1176 | 1173 | ||
1177 | if (inode) | ||
1178 | iput(inode); | ||
1179 | |||
1180 | return err; | 1174 | return err; |
1181 | 1175 | ||
1182 | } | 1176 | } |