ocfs2: delete unnecessary checks before three function calls
authorMarkus Elfring <elfring@users.sourceforge.net>
Tue, 14 Apr 2015 22:42:42 +0000 (15:42 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 14 Apr 2015 23:48:56 +0000 (16:48 -0700)
kfree(), ocfs2_free_path() and __ocfs2_free_slot_info() test whether their
argument is NULL and then return immediately.  Thus the test around their
calls is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
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
fs/ocfs2/slot_map.c
fs/ocfs2/stack_user.c

index 044158bd22be1c307aa046d9863a0684dfde9824..fdab27c9be999e387b984f18d732fa1b72202d0d 100644 (file)
@@ -3453,8 +3453,7 @@ static int ocfs2_merge_rec_right(struct ocfs2_path *left_path,
                                           subtree_index);
        }
 out:
-       if (right_path)
-               ocfs2_free_path(right_path);
+       ocfs2_free_path(right_path);
        return ret;
 }
 
@@ -3647,8 +3646,7 @@ static int ocfs2_merge_rec_left(struct ocfs2_path *right_path,
                                                   right_path, subtree_index);
        }
 out:
-       if (left_path)
-               ocfs2_free_path(left_path);
+       ocfs2_free_path(left_path);
        return ret;
 }
 
@@ -4431,11 +4429,8 @@ ocfs2_figure_merge_contig_type(struct ocfs2_extent_tree *et,
        }
 
 out:
-       if (left_path)
-               ocfs2_free_path(left_path);
-       if (right_path)
-               ocfs2_free_path(right_path);
-
+       ocfs2_free_path(left_path);
+       ocfs2_free_path(right_path);
        return ret;
 }
 
@@ -6996,9 +6991,7 @@ out_commit:
 out:
        if (data_ac)
                ocfs2_free_alloc_context(data_ac);
-       if (pages)
-               kfree(pages);
-
+       kfree(pages);
        return ret;
 }
 
index d5493e361a381b12a9d57a779294fb172fb963b3..c5e530a9d1b12b40e2e2fc1076b03139824e6f6e 100644 (file)
@@ -452,7 +452,7 @@ int ocfs2_init_slot_info(struct ocfs2_super *osb)
 
        osb->slot_info = (struct ocfs2_slot_info *)si;
 bail:
-       if (status < 0 && si)
+       if (status < 0)
                __ocfs2_free_slot_info(si);
 
        return status;
index 720aa389e0eae60c2b0d01a7038b73c268932d5a..c3b7807c65d6f72812a9bfa6512a0e9e705d3165 100644 (file)
@@ -1063,7 +1063,7 @@ static int user_cluster_connect(struct ocfs2_cluster_connection *conn)
        }
 
 out:
-       if (rc && lc)
+       if (rc)
                kfree(lc);
        return rc;
 }