From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> cvtnum() returns a signed long long, so the type of 'len' should be a signed type so that a user entering a negative length doesn't produce some huge positive integer. The negative len check demands it anyway. Coverity-id: 1435895 Fixes: 25b4549c8b54134106cff094aa098eab9e86eee7 Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> --- io/copy_file_range.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/io/copy_file_range.c b/io/copy_file_range.c index ebd1ec76..4d0770ef 100644 --- a/io/copy_file_range.c +++ b/io/copy_file_range.c @@ -91,7 +91,7 @@ copy_range_f(int argc, char **argv) { long long src = 0; long long dst = 0; - size_t len = 0; + long long len = 0; int opt; int ret; int fd; -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html