On 2015/9/7 14:32, Norton.Zhu wrote:
> If ocfs2_is_overwrite failed, ocfs2_direct_IO_write may still return success
> to the caller.
> changelog:
> Using mlog_errno(ret) looks better than mlog_errno(is_overwrite)
>
> Signed-off-by: Norton.Zhu <norton....@huawei.com>
Reviewed-by: Joseph Qi <joseph...@huawei.com>
> ---
> aops.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/aops.c b/aops.c
> index 0f5fd9d..01bbc56 100644
> --- a/aops.c
> +++ b/aops.c
> @@ -846,7 +846,8 @@ 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;
> + mlog_errno(ret);
> ocfs2_inode_unlock(inode, 1);
> goto clean_orphan;
> }
>
_______________________________________________
Ocfs2-devel mailing list
Ocfs2-devel@oss.oracle.com
https://oss.oracle.com/mailman/listinfo/ocfs2-devel
