ocfs2: return error when ocfs2_figure_merge_contig_type() fails
authorXue jiufei <xuejiufei@huawei.com>
Wed, 24 Jun 2015 23:55:07 +0000 (16:55 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 25 Jun 2015 00:49:39 +0000 (17:49 -0700)
ocfs2_figure_merge_contig_type() still returns CONTIG_NONE when some error
occurs which will cause an unpredictable error.  So return a proper errno
when ocfs2_figure_merge_contig_type() fails.

Signed-off-by: joyce.xue <xuejiufei@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>
fs/ocfs2/alloc.c

index 2d7f76e52c379cf04b7f820c610c171e491d4830..ea4c822531cc9a86c9c75527dfd4b3fd79a16f23 100644 (file)
@@ -4311,13 +4311,13 @@ out:
        return ret;
 }
 
-static enum ocfs2_contig_type
-ocfs2_figure_merge_contig_type(struct ocfs2_extent_tree *et,
+static int ocfs2_figure_merge_contig_type(struct ocfs2_extent_tree *et,
                               struct ocfs2_path *path,
                               struct ocfs2_extent_list *el, int index,
-                              struct ocfs2_extent_rec *split_rec)
+                              struct ocfs2_extent_rec *split_rec,
+                              struct ocfs2_merge_ctxt *ctxt)
 {
-       int status;
+       int status = 0;
        enum ocfs2_contig_type ret = CONTIG_NONE;
        u32 left_cpos, right_cpos;
        struct ocfs2_extent_rec *rec = NULL;
@@ -4336,8 +4336,11 @@ ocfs2_figure_merge_contig_type(struct ocfs2_extent_tree *et,
 
                if (left_cpos != 0) {
                        left_path = ocfs2_new_path_from_path(path);
-                       if (!left_path)
+                       if (!left_path) {
+                               status = -ENOMEM;
+                               mlog_errno(status);
                                goto exit;
+                       }
 
                        status = ocfs2_find_path(et->et_ci, left_path,
                                                 left_cpos);
@@ -4392,8 +4395,11 @@ ocfs2_figure_merge_contig_type(struct ocfs2_extent_tree *et,
                        goto free_left_path;
 
                right_path = ocfs2_new_path_from_path(path);
-               if (!right_path)
+               if (!right_path) {
+                       status = -ENOMEM;
+                       mlog_errno(status);
                        goto free_left_path;
+               }
 
                status = ocfs2_find_path(et->et_ci, right_path, right_cpos);
                if (status)
@@ -4433,7 +4439,10 @@ free_right_path:
 free_left_path:
        ocfs2_free_path(left_path);
 exit:
-       return ret;
+       if (status == 0)
+               ctxt->c_contig_type = ret;
+
+       return status;
 }
 
 static void ocfs2_figure_contig_type(struct ocfs2_extent_tree *et,
@@ -5039,9 +5048,14 @@ int ocfs2_split_extent(handle_t *handle,
                goto out;
        }
 
-       ctxt.c_contig_type = ocfs2_figure_merge_contig_type(et, path, el,
-                                                           split_index,
-                                                           split_rec);
+       ret = ocfs2_figure_merge_contig_type(et, path, el,
+                                            split_index,
+                                            split_rec,
+                                            &ctxt);
+       if (ret) {
+               mlog_errno(ret);
+               goto out;
+       }
 
        /*
         * The core merge / split code wants to know how much room is