Re: [Qemu-devel] [PATCH v3 3/7] block: fix data type casting for crypto payload offset
On 09/12/2017 06:28 AM, Daniel P. Berrange wrote: > The crypto APIs report the offset of the data payload as an uint64_t > type, but the block driver is casting to size_t or ssize_t which will > potentially truncate. > > Most of the block APIs use int64_t for offsets meanwhile, so even if > using uint64_t in the crypto block driver we are still at risk of > truncation. > > Change the block crypto driver to use uint64_t, but add asserts that > the value is less than INT64_MAX. > > Signed-off-by: Daniel P. Berrange> --- > block/crypto.c | 13 + > 1 file changed, 9 insertions(+), 4 deletions(-) > Reviewed-by: Eric Blake -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org signature.asc Description: OpenPGP digital signature
Re: [Qemu-devel] [PATCH v3 3/7] block: fix data type casting for crypto payload offset
On 2017-09-12 13:28, Daniel P. Berrange wrote: > The crypto APIs report the offset of the data payload as an uint64_t > type, but the block driver is casting to size_t or ssize_t which will > potentially truncate. > > Most of the block APIs use int64_t for offsets meanwhile, so even if > using uint64_t in the crypto block driver we are still at risk of > truncation. > > Change the block crypto driver to use uint64_t, but add asserts that > the value is less than INT64_MAX. > > Signed-off-by: Daniel P. Berrange> --- > block/crypto.c | 13 + > 1 file changed, 9 insertions(+), 4 deletions(-) Reviewed-by: Max Reitz signature.asc Description: OpenPGP digital signature