diff options
| author | Peter Maydell | 2022-07-21 13:52:07 +0200 |
|---|---|---|
| committer | Dr. David Alan Gilbert | 2022-08-02 17:46:52 +0200 |
| commit | 4bcb7de072fd8ab9de40b24aac00c7b6659f7f7d (patch) | |
| tree | f3255476c4b51d8e9c1f61c49ac4d255674edd9f /python/Pipfile.lock | |
| parent | migration: Assert that migrate_multifd_compression() returns an in-range value (diff) | |
| download | qemu-4bcb7de072fd8ab9de40b24aac00c7b6659f7f7d.tar.gz qemu-4bcb7de072fd8ab9de40b24aac00c7b6659f7f7d.tar.xz qemu-4bcb7de072fd8ab9de40b24aac00c7b6659f7f7d.zip | |
migration: Define BLK_MIG_BLOCK_SIZE as unsigned long long
When we use BLK_MIG_BLOCK_SIZE in expressions like
block_mig_state.submitted * BLK_MIG_BLOCK_SIZE, this multiplication
is done as 32 bits, because both operands are 32 bits. Coverity
complains about possible overflows because we then accumulate that
into a 64 bit variable.
Define BLK_MIG_BLOCK_SIZE as unsigned long long using the ULL suffix.
The only two current uses of it with this problem are both in
block_save_pending(), so we could just cast to uint64_t there, but
using the ULL suffix is simpler and ensures that we don't
accidentally introduce new variants of the same issue in future.
Resolves: Coverity CID 1487136, 1487175
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20220721115207.729615-3-peter.maydell@linaro.org>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Diffstat (limited to 'python/Pipfile.lock')
0 files changed, 0 insertions, 0 deletions