diff options
author | Daeseok Youn <daeseok.youn@gmail.com> | 2015-04-14 18:43:30 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-04-14 19:48:58 -0400 |
commit | 7c01ad8fe7c159d7e1ddbd8e586d4d0dfed7ab3d (patch) | |
tree | 47c24e57e37ab08fafef2ecc1e1d2c2491259182 /fs/ocfs2 | |
parent | a47726bcf299db6b5743d574df36c423263b4e65 (diff) |
ocfs2: remove goto statement in ocfs2_check_dir_for_entry()
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
Reviewed-by: Joseph Qi <joseph.qi@huawei.com>
Cc: Mark Fasheh <mfasheh@suse.com>
Cc: Joel Becker <jlbec@evilplan.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/ocfs2')
-rw-r--r-- | fs/ocfs2/dir.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/fs/ocfs2/dir.c b/fs/ocfs2/dir.c index b08050bd3f2e..2241a19b9335 100644 --- a/fs/ocfs2/dir.c +++ b/fs/ocfs2/dir.c | |||
@@ -2047,22 +2047,19 @@ int ocfs2_check_dir_for_entry(struct inode *dir, | |||
2047 | const char *name, | 2047 | const char *name, |
2048 | int namelen) | 2048 | int namelen) |
2049 | { | 2049 | { |
2050 | int ret; | 2050 | int ret = 0; |
2051 | struct ocfs2_dir_lookup_result lookup = { NULL, }; | 2051 | struct ocfs2_dir_lookup_result lookup = { NULL, }; |
2052 | 2052 | ||
2053 | trace_ocfs2_check_dir_for_entry( | 2053 | trace_ocfs2_check_dir_for_entry( |
2054 | (unsigned long long)OCFS2_I(dir)->ip_blkno, namelen, name); | 2054 | (unsigned long long)OCFS2_I(dir)->ip_blkno, namelen, name); |
2055 | 2055 | ||
2056 | ret = -EEXIST; | 2056 | if (ocfs2_find_entry(name, namelen, dir, &lookup) == 0) { |
2057 | if (ocfs2_find_entry(name, namelen, dir, &lookup) == 0) | 2057 | ret = -EEXIST; |
2058 | goto bail; | 2058 | mlog_errno(ret); |
2059 | } | ||
2059 | 2060 | ||
2060 | ret = 0; | ||
2061 | bail: | ||
2062 | ocfs2_free_dir_lookup_result(&lookup); | 2061 | ocfs2_free_dir_lookup_result(&lookup); |
2063 | 2062 | ||
2064 | if (ret) | ||
2065 | mlog_errno(ret); | ||
2066 | return ret; | 2063 | return ret; |
2067 | } | 2064 | } |
2068 | 2065 | ||