aboutsummaryrefslogtreecommitdiff
path: root/block
diff options
context:
space:
mode:
authorStefan Weil <weil@mail.berlios.de>2011-02-04 21:01:16 +0100
committerKevin Wolf <kwolf@redhat.com>2011-02-07 10:07:25 +0100
commit4f3669ea5bd73ade0dce5f1155cb9ad9788fd54c (patch)
tree13a5be2d5cf1057492acc47c996cdf291b339244 /block
parent42af9c30ea9a963ce604ac96230fde2f987634db (diff)
downloadqemu-4f3669ea5bd73ade0dce5f1155cb9ad9788fd54c.zip
qemu-4f3669ea5bd73ade0dce5f1155cb9ad9788fd54c.tar.gz
qemu-4f3669ea5bd73ade0dce5f1155cb9ad9788fd54c.tar.bz2
block/vdi: Fix wrong size in conditionally used memset, memcmp
Error report from cppcheck: block/vdi.c:122: error: Using sizeof for array given as function argument returns the size of pointer. block/vdi.c:128: error: Using sizeof for array given as function argument returns the size of pointer. Fix both by setting the correct size. The buggy code is only used when QEMU is build without uuid support. The bug is not critical, so there is no urgent need to apply it to old versions of QEMU. Cc: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Stefan Weil <weil@mail.berlios.de> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block')
-rw-r--r--block/vdi.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/block/vdi.c b/block/vdi.c
index ab8f70f..116b25b 100644
--- a/block/vdi.c
+++ b/block/vdi.c
@@ -119,13 +119,13 @@ void uuid_unparse(const uuid_t uu, char *out);
#if !defined(CONFIG_UUID)
void uuid_generate(uuid_t out)
{
- memset(out, 0, sizeof(out));
+ memset(out, 0, sizeof(uuid_t));
}
int uuid_is_null(const uuid_t uu)
{
uuid_t null_uuid = { 0 };
- return memcmp(uu, null_uuid, sizeof(uu)) == 0;
+ return memcmp(uu, null_uuid, sizeof(uuid_t)) == 0;
}
void uuid_unparse(const uuid_t uu, char *out)