Patchwork spice-qemu-char.c: Use correct printf format char for ssize_t

login
register
mail settings
Submitter Peter Maydell
Date Aug. 9, 2011, 10:04 p.m.
Message ID <1312927475-22082-1-git-send-email-peter.maydell@linaro.org>
Download mbox | patch
Permalink /patch/109299/
State New
Headers show

Comments

Peter Maydell - Aug. 9, 2011, 10:04 p.m.
Use the correct printf format string character (%z) for ssize_t.
This fixes a compile failure on 32 bit Linux with spice enabled.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---

I note that this seems to have been present since the file
was introduced, so presumably nobody up til now has attempted
to compile qemu with spice enabled on a 32 bit box...

This is present in 0.15 (although not fatal since we don't enable
Werror on release builds); is it worth including compile warning
fixes in 0.15.1 ?

 spice-qemu-char.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Peter Maydell - Aug. 18, 2011, 1:06 a.m.
Ping?

-- PMM

On 9 August 2011 23:04, Peter Maydell <peter.maydell@linaro.org> wrote:
> Use the correct printf format string character (%z) for ssize_t.
> This fixes a compile failure on 32 bit Linux with spice enabled.
>
> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
> ---
>
> I note that this seems to have been present since the file
> was introduced, so presumably nobody up til now has attempted
> to compile qemu with spice enabled on a 32 bit box...
>
> This is present in 0.15 (although not fatal since we don't enable
> Werror on release builds); is it worth including compile warning
> fixes in 0.15.1 ?
>
>  spice-qemu-char.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/spice-qemu-char.c b/spice-qemu-char.c
> index 95bf6b6..7ec1a9b 100644
> --- a/spice-qemu-char.c
> +++ b/spice-qemu-char.c
> @@ -45,7 +45,7 @@ static int vmc_write(SpiceCharDeviceInstance *sin, const uint8_t *buf, int len)
>         p += last_out;
>     }
>
> -    dprintf(scd, 3, "%s: %lu/%zd\n", __func__, out, len + out);
> +    dprintf(scd, 3, "%s: %zu/%zd\n", __func__, out, len + out);
>     trace_spice_vmc_write(out, len + out);
>     return out;
>  }
> --
> 1.7.4.1
Gerd Hoffmann - Aug. 22, 2011, 6:44 a.m.
On 08/18/11 03:06, Peter Maydell wrote:
> Ping?

Added to spice patch queue.

thanks,
   Gerd

[ starting to catch up kvm forum mail backlog ... ]

Patch

diff --git a/spice-qemu-char.c b/spice-qemu-char.c
index 95bf6b6..7ec1a9b 100644
--- a/spice-qemu-char.c
+++ b/spice-qemu-char.c
@@ -45,7 +45,7 @@  static int vmc_write(SpiceCharDeviceInstance *sin, const uint8_t *buf, int len)
         p += last_out;
     }
 
-    dprintf(scd, 3, "%s: %lu/%zd\n", __func__, out, len + out);
+    dprintf(scd, 3, "%s: %zu/%zd\n", __func__, out, len + out);
     trace_spice_vmc_write(out, len + out);
     return out;
 }