From patchwork Mon May 7 08:21:06 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Miklos Szeredi X-Patchwork-Id: 10383421 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id A9358602D8 for ; Mon, 7 May 2018 08:22:23 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 9E0CA27031 for ; Mon, 7 May 2018 08:22:23 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 92FD0287F3; Mon, 7 May 2018 08:22:23 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.9 required=2.0 tests=BAYES_00, MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 3BBF827031 for ; Mon, 7 May 2018 08:22:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752117AbeEGIWV (ORCPT ); Mon, 7 May 2018 04:22:21 -0400 Received: from mail-wm0-f66.google.com ([74.125.82.66]:37693 "EHLO mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751998AbeEGIVQ (ORCPT ); Mon, 7 May 2018 04:21:16 -0400 Received: by mail-wm0-f66.google.com with SMTP id l1-v6so13754711wmb.2 for ; Mon, 07 May 2018 01:21:15 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=zaKfSZy6t/Zyw7AuRBrwicvBnUQ/4uG9MRkSTNZ5qaY=; b=YnaxK377dKy7e0VNudnPDLTcfLxhOV4llieyqq9xbIlY+5do8hHUNHryqF+hLMlyRG DQRn6K9yWLqsuLmipjbYDccUgt8JnUeHoRbGLaUive9Ey7iPfQeLr5qB/cKqlqmSRXU8 2XA+4dX5j189EpElYzj8DZTBpgHFuydbhrG5awXB+SIcvzCwg1YIQ1pswiZKABDQJdCi MSJfA1hDhbUWAiF9L/jV1pB20pdoqQJrMVmF/fm7MIv+7Maol/DWS+ZPdtpvgS5maAH0 jHo67W/UmcNBB4wn6vlnFPUapZICKa9HgZ3TsQR0k3k6P/plvSVjQKaPzoC8xz68mONL oMQw== X-Gm-Message-State: ALQs6tDl8yBosySYy8KbTcp5mZcLR5SwHqCfITBXW+nJV9txwV517sZ5 pggXoZX/6nyJekzdzfeqWbY4lkUa9uc= X-Google-Smtp-Source: AB8JxZrkALPPA6fOFNUW578SwTjvGqScD9m9Gk6mW/8swsEJmlTuFUFkYoq7puD92GEt3OYvFf/XGQ== X-Received: by 10.28.134.17 with SMTP id i17mr118895wmd.107.1525681274914; Mon, 07 May 2018 01:21:14 -0700 (PDT) Received: from veci.piliscsaba.redhat.com (C2B0E321.catv.pool.telekom.hu. [194.176.227.33]) by smtp.gmail.com with ESMTPSA id w6-v6sm26991776wra.16.2018.05.07.01.21.13 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 07 May 2018 01:21:14 -0700 (PDT) From: Miklos Szeredi To: Al Viro Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Amir Goldstein Subject: [PATCH 1/3] vfs: dedpue: return s64 Date: Mon, 7 May 2018 10:21:06 +0200 Message-Id: <20180507082108.28186-2-mszeredi@redhat.com> X-Mailer: git-send-email 2.14.3 In-Reply-To: <20180507082108.28186-1-mszeredi@redhat.com> References: <20180507082108.28186-1-mszeredi@redhat.com> Sender: linux-fsdevel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP f_op->dedupe_file_range() gets a u64 length to dedup and returns an ssize_t actual length deduped. This breaks badly on 32bit archs since the returned length will be truncated and possibly overflow into the sign bit (xfs and ocfs2 are affected, btrfs limits actual length to 16MiB). Returning s64 should be good, since clone_verify_area() makes sure that the supplied length doesn't overflow. Signed-off-by: Miklos Szeredi --- fs/btrfs/ctree.h | 4 ++-- fs/btrfs/ioctl.c | 6 +++--- fs/ocfs2/file.c | 10 +++++----- fs/read_write.c | 2 +- fs/xfs/xfs_file.c | 2 +- include/linux/fs.h | 2 +- 6 files changed, 13 insertions(+), 13 deletions(-) diff --git a/fs/btrfs/ctree.h b/fs/btrfs/ctree.h index 2771cc56a622..019a25962ac8 100644 --- a/fs/btrfs/ctree.h +++ b/fs/btrfs/ctree.h @@ -3269,8 +3269,8 @@ void btrfs_get_block_group_info(struct list_head *groups_list, struct btrfs_ioctl_space_info *space); void update_ioctl_balance_args(struct btrfs_fs_info *fs_info, int lock, struct btrfs_ioctl_balance_args *bargs); -ssize_t btrfs_dedupe_file_range(struct file *src_file, u64 loff, u64 olen, - struct file *dst_file, u64 dst_loff); +s64 btrfs_dedupe_file_range(struct file *src_file, u64 loff, u64 olen, + struct file *dst_file, u64 dst_loff); /* file.c */ int __init btrfs_auto_defrag_init(void); diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c index 632e26d6f7ce..b6a1df6bb895 100644 --- a/fs/btrfs/ioctl.c +++ b/fs/btrfs/ioctl.c @@ -3194,13 +3194,13 @@ static int btrfs_extent_same(struct inode *src, u64 loff, u64 olen, #define BTRFS_MAX_DEDUPE_LEN SZ_16M -ssize_t btrfs_dedupe_file_range(struct file *src_file, u64 loff, u64 olen, - struct file *dst_file, u64 dst_loff) +s64 btrfs_dedupe_file_range(struct file *src_file, u64 loff, u64 olen, + struct file *dst_file, u64 dst_loff) { struct inode *src = file_inode(src_file); struct inode *dst = file_inode(dst_file); u64 bs = BTRFS_I(src)->root->fs_info->sb->s_blocksize; - ssize_t res; + int res; if (olen > BTRFS_MAX_DEDUPE_LEN) olen = BTRFS_MAX_DEDUPE_LEN; diff --git a/fs/ocfs2/file.c b/fs/ocfs2/file.c index 6ee94bc23f5b..cf3732df4c2e 100644 --- a/fs/ocfs2/file.c +++ b/fs/ocfs2/file.c @@ -2537,11 +2537,11 @@ static int ocfs2_file_clone_range(struct file *file_in, len, false); } -static ssize_t ocfs2_file_dedupe_range(struct file *src_file, - u64 loff, - u64 len, - struct file *dst_file, - u64 dst_loff) +static s64 ocfs2_file_dedupe_range(struct file *src_file, + u64 loff, + u64 len, + struct file *dst_file, + u64 dst_loff) { int error; diff --git a/fs/read_write.c b/fs/read_write.c index c4eabbfc90df..6a79c7ec2bb2 100644 --- a/fs/read_write.c +++ b/fs/read_write.c @@ -1976,7 +1976,7 @@ int vfs_dedupe_file_range(struct file *file, struct file_dedupe_range *same) u16 count = same->dest_count; struct file *dst_file; loff_t dst_off; - ssize_t deduped; + s64 deduped; if (!(file->f_mode & FMODE_READ)) return -EINVAL; diff --git a/fs/xfs/xfs_file.c b/fs/xfs/xfs_file.c index 299aee4b7b0b..d06dd1557d0e 100644 --- a/fs/xfs/xfs_file.c +++ b/fs/xfs/xfs_file.c @@ -868,7 +868,7 @@ xfs_file_clone_range( len, false); } -STATIC ssize_t +STATIC s64 xfs_file_dedupe_range( struct file *src_file, u64 loff, diff --git a/include/linux/fs.h b/include/linux/fs.h index 760d8da1b6c7..6feb121ae48c 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h @@ -1738,7 +1738,7 @@ struct file_operations { loff_t, size_t, unsigned int); int (*clone_file_range)(struct file *, loff_t, struct file *, loff_t, u64); - ssize_t (*dedupe_file_range)(struct file *, u64, u64, struct file *, + s64 (*dedupe_file_range)(struct file *, u64, u64, struct file *, u64); } __randomize_layout;