aboutsummaryrefslogtreecommitdiff
path: root/block
diff options
context:
space:
mode:
authorStefan Weil <weil@mail.berlios.de>2010-09-24 21:02:05 +0200
committerKevin Wolf <kwolf@redhat.com>2010-10-22 14:49:35 +0200
commita77cffe7e916f4dd28f2048982ea2e0d98143b11 (patch)
treef577912f7ee2f342a4b8719e8e7372effb7618ed /block
parent191c2890ddc9626d35c9e432494e1699a8181025 (diff)
downloadqemu-a77cffe7e916f4dd28f2048982ea2e0d98143b11.zip
qemu-a77cffe7e916f4dd28f2048982ea2e0d98143b11.tar.gz
qemu-a77cffe7e916f4dd28f2048982ea2e0d98143b11.tar.bz2
block: Use GCC_FMT_ATTR and fix a format error
Adding the gcc format attribute detects a format bug which is fixed here. v2: Don't use type cast. BDRV_SECTOR_SIZE is unsigned long long, so %lld should be the correct format specifier. Cc: Blue Swirl <blauwirbel@gmail.com> 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/blkverify.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/block/blkverify.c b/block/blkverify.c
index 8083464..b2a12fe 100644
--- a/block/blkverify.c
+++ b/block/blkverify.c
@@ -53,7 +53,8 @@ static AIOPool blkverify_aio_pool = {
.cancel = blkverify_aio_cancel,
};
-static void blkverify_err(BlkverifyAIOCB *acb, const char *fmt, ...)
+static void GCC_FMT_ATTR(2, 3) blkverify_err(BlkverifyAIOCB *acb,
+ const char *fmt, ...)
{
va_list ap;
@@ -299,7 +300,7 @@ static void blkverify_verify_readv(BlkverifyAIOCB *acb)
{
ssize_t offset = blkverify_iovec_compare(acb->qiov, &acb->raw_qiov);
if (offset != -1) {
- blkverify_err(acb, "contents mismatch in sector %ld",
+ blkverify_err(acb, "contents mismatch in sector %lld",
acb->sector_num + (offset / BDRV_SECTOR_SIZE));
}
}