summary refs log tree commit diff stats
path: root/scripts/qapi/types.py
diff options
context:
space:
mode:
authorAlberto Garcia <berto@igalia.com>2018-05-28 17:01:28 +0200
committerKevin Wolf <kwolf@redhat.com>2018-05-29 20:09:17 +0200
commit7af5eea9b34ffb7a9a9fc25ba71998a02b76e159 (patch)
tree9501a0b5d3c2e3898b9d21ae93e72875415e65fc /scripts/qapi/types.py
parente609fa71e89c81fbe2670411be62da95dfb093e0 (diff)
downloadfocaccia-qemu-7af5eea9b34ffb7a9a9fc25ba71998a02b76e159.tar.gz
focaccia-qemu-7af5eea9b34ffb7a9a9fc25ba71998a02b76e159.zip
qcow2: Fix Coverity warning when calculating the refcount cache size
MIN_REFCOUNT_CACHE_SIZE is 4 and the cluster size is guaranteed to be
at most 2MB, so the minimum refcount cache size (in bytes) is always
going to fit in a 32-bit integer.

Coverity doesn't know that, and since we're storing the result in a
uint64_t (*refcount_cache_size) it thinks that we need the 64 bits and
that we probably want to do a 64-bit multiplication to prevent the
result from being truncated.

This is a false positive in this case, but it's a fair warning.
We could do a 64-bit multiplication to get rid of it, but since we
know that a 32-bit variable is enough to store this value let's simply
reuse min_refcount_cache, make it a normal int and stop doing casts.

Reported-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Alberto Garcia <berto@igalia.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'scripts/qapi/types.py')
0 files changed, 0 insertions, 0 deletions