diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-04-22 05:39:29 -0400 |
|---|---|---|
| committer | Joel Becker <joel.becker@oracle.com> | 2010-04-23 17:42:06 -0400 |
| commit | 0350cb078f5035716ebdad4ad4709d02fe466a8a (patch) | |
| tree | 17b2166b5c71e22571b0c244e130325d33bb77aa | |
| parent | 79681842e160c3211eeeb47ea31b061038d1e41e (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.c | 1 |
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 | } |
