diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-04-29 18:05:57 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-04-29 18:54:27 -0400 |
commit | 7ebab4536958b05f65b71ec312073acf5d66578d (patch) | |
tree | 46d9f64f5547944b5f367c7d1f4eee3bf31082bc /fs/ocfs2/ioctl.c | |
parent | b3e0767abcd6eda32cdfd9c6a6b9e53b4ddf6d4d (diff) |
ocfs2: fix error return code in ocfs2_info_handle_freefrag()
Fix to return a negative error code from the error handling case instead
of 0, as returned elsewhere in this function.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Mark Fasheh <mfasheh@suse.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/ocfs2/ioctl.c')
-rw-r--r-- | fs/ocfs2/ioctl.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/ocfs2/ioctl.c b/fs/ocfs2/ioctl.c index 6cab301a568d..0c60ef2d8056 100644 --- a/fs/ocfs2/ioctl.c +++ b/fs/ocfs2/ioctl.c | |||
@@ -706,8 +706,10 @@ int ocfs2_info_handle_freefrag(struct inode *inode, | |||
706 | 706 | ||
707 | o2info_set_request_filled(&oiff->iff_req); | 707 | o2info_set_request_filled(&oiff->iff_req); |
708 | 708 | ||
709 | if (o2info_to_user(*oiff, req)) | 709 | if (o2info_to_user(*oiff, req)) { |
710 | status = -EFAULT; | ||
710 | goto bail; | 711 | goto bail; |
712 | } | ||
711 | 713 | ||
712 | status = 0; | 714 | status = 0; |
713 | bail: | 715 | bail: |