aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2010-04-22 05:39:29 -0400
committerJoel Becker <joel.becker@oracle.com>2010-04-23 17:42:06 -0400
commit0350cb078f5035716ebdad4ad4709d02fe466a8a (patch)
tree17b2166b5c71e22571b0c244e130325d33bb77aa
parent79681842e160c3211eeeb47ea31b061038d1e41e (diff)
ocfs2: potential ERR_PTR dereference on error paths
If "handle" is non null at the end of the function then we assume it's a valid pointer and pass it to ocfs2_commit_trans(); Signed-off-by: Dan Carpenter <error27@gmail.com> Cc: <stable@kernel.org> Signed-off-by: Joel Becker <joel.becker@oracle.com>
-rw-r--r--fs/ocfs2/inode.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/ocfs2/inode.c b/fs/ocfs2/inode.c
index ab207901d32a..23c254e26018 100644
--- a/fs/ocfs2/inode.c
+++ b/fs/ocfs2/inode.c
@@ -559,6 +559,7 @@ static int ocfs2_truncate_for_delete(struct ocfs2_super *osb,
559 handle = ocfs2_start_trans(osb, OCFS2_INODE_UPDATE_CREDITS); 559 handle = ocfs2_start_trans(osb, OCFS2_INODE_UPDATE_CREDITS);
560 if (IS_ERR(handle)) { 560 if (IS_ERR(handle)) {
561 status = PTR_ERR(handle); 561 status = PTR_ERR(handle);
562 handle = NULL;
562 mlog_errno(status); 563 mlog_errno(status);
563 goto out; 564 goto out;
564 } 565 }