]> www.infradead.org Git - users/hch/xfsprogs.git/commit
xfs_io: fix pwrite/pread length truncation on 32-bit systems
authorDarrick J. Wong <darrick.wong@oracle.com>
Thu, 30 Jan 2020 18:40:54 +0000 (13:40 -0500)
committerEric Sandeen <sandeen@sandeen.net>
Thu, 30 Jan 2020 18:40:54 +0000 (13:40 -0500)
commit4eafc4d3fdf088dd66b64ff6718c716ed84d457f
treef8c144123148a1730d634cc1926d68be1dc4aaba
parent2dd6310832e423151c5bf6cea95e3aa45170627d
xfs_io: fix pwrite/pread length truncation on 32-bit systems

The pwrite and pread commands in xfs_io accept an operation length that
can be any quantity that fits in a long long int; and loops to handle
the cases where the operation length is larger than the IO buffer.

Weirdly, the do_ functions contain code to shorten the operation to the
IO buffer size but the @count parameter is size_t, which means that for
a large argument on a 32-bit system, we rip off the upper bits of the
length, turning your 8GB write into a 0 byte write, which does nothing.

This was found by running generic/175 and observing that the 8G test
file it creates has zero length after the operation:

wrote 0/8589934592 bytes at offset 0
0.000000 bytes, 0 ops; 0.0001 sec (0.000000 bytes/sec and 0.0000 ops/sec)

Fix this by pushing long long count all the way through the call stack.

Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
io/pread.c
io/pwrite.c