summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColy Li <coyli@suse.de>2008-11-17 12:38:22 +0800
committerMark Fasheh <mfasheh@suse.com>2008-12-01 14:46:55 -0800
commit07d9a3954a68764aefe16855bcd0f86deeb5c825 (patch)
tree16721b539e67d78eee336f124a89b9a089175773
parenta2eee69b814854095ed835a6eb64b8efc220cd6a (diff)
downloadlinux-07d9a3954a68764aefe16855bcd0f86deeb5c825.tar.bz2
ocfs2: fix return value set in init_dlmfs_fs()
In init_dlmfs_fs(), if calling kmem_cache_create() failed, the code will use return value from calling bdi_init(). The correct behavior should be set status as -ENOMEM before going to "bail:". Signed-off-by: Coly Li <coyli@suse.de> Acked-by: Sunil Mushran <sunil.mushran@oracle.com> Signed-off-by: Mark Fasheh <mfasheh@suse.com>
-rw-r--r--fs/ocfs2/dlm/dlmfs.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/ocfs2/dlm/dlmfs.c b/fs/ocfs2/dlm/dlmfs.c
index 533a789c3ef8..ba962d71b34d 100644
--- a/fs/ocfs2/dlm/dlmfs.c
+++ b/fs/ocfs2/dlm/dlmfs.c
@@ -608,8 +608,10 @@ static int __init init_dlmfs_fs(void)
0, (SLAB_HWCACHE_ALIGN|SLAB_RECLAIM_ACCOUNT|
SLAB_MEM_SPREAD),
dlmfs_init_once);
- if (!dlmfs_inode_cache)
+ if (!dlmfs_inode_cache) {
+ status = -ENOMEM;
goto bail;
+ }
cleanup_inode = 1;
user_dlm_worker = create_singlethread_workqueue("user_dlm");