summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNorton.Zhu <norton.zhu@huawei.com>2015-11-05 18:43:52 -0800
committerLinus Torvalds <torvalds@linux-foundation.org>2015-11-05 19:34:48 -0800
commitd162eaad7726e5f10a2b5813bdfac9d55c8eba69 (patch)
tree1b2813ce95159ad838a02bd5e5d6e5b9c7255e74
parentce4f2fd7eacd84d78530fb97621621ae50d167f1 (diff)
downloadop-kernel-dev-d162eaad7726e5f10a2b5813bdfac9d55c8eba69.zip
op-kernel-dev-d162eaad7726e5f10a2b5813bdfac9d55c8eba69.tar.gz
ocfs2_direct_IO_write() misses ocfs2_is_overwrite() error code
If ocfs2_is_overwrite failed, ocfs2_direct_IO_write mays till return success to the caller. Signed-off-by: Norton.Zhu <norton.zhu@huawei.com> Cc: Mark Fasheh <mfasheh@suse.de> Cc: Joel Becker <jlbec@evilplan.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--fs/ocfs2/aops.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/ocfs2/aops.c b/fs/ocfs2/aops.c
index 64b11d9..f04914c 100644
--- a/fs/ocfs2/aops.c
+++ b/fs/ocfs2/aops.c
@@ -864,6 +864,7 @@ static ssize_t ocfs2_direct_IO_write(struct kiocb *iocb,
is_overwrite = ocfs2_is_overwrite(osb, inode, offset);
if (is_overwrite < 0) {
mlog_errno(is_overwrite);
+ ret = is_overwrite;
ocfs2_inode_unlock(inode, 1);
goto clean_orphan;
}
OpenPOWER on IntegriCloud