diff options
author | Coly Li <coyli@suse.de> | 2008-11-17 12:38:22 +0800 |
---|---|---|
committer | Mark Fasheh <mfasheh@suse.com> | 2008-12-01 14:46:55 -0800 |
commit | 07d9a3954a68764aefe16855bcd0f86deeb5c825 (patch) | |
tree | 16721b539e67d78eee336f124a89b9a089175773 /fs/ocfs2/dlm | |
parent | a2eee69b814854095ed835a6eb64b8efc220cd6a (diff) | |
download | linux-07d9a3954a68764aefe16855bcd0f86deeb5c825.tar.gz linux-07d9a3954a68764aefe16855bcd0f86deeb5c825.tar.bz2 linux-07d9a3954a68764aefe16855bcd0f86deeb5c825.zip |
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>
Diffstat (limited to 'fs/ocfs2/dlm')
-rw-r--r-- | fs/ocfs2/dlm/dlmfs.c | 4 |
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"); |