g_object_new is a variadic function which take property values.
As compiler cannot check if these property values are correct
make sure they are using casts.
This actully fix a crash in reds.c for 32 bit architectures.

Based on a patch by Frediano Ziglio <fzig...@redhat.com>
Signed-off-by: Francois Gouget <fgou...@codeweavers.com>
---

I ran into this crash so I thought I'd resubmit this patch so nobody 
else has to dig for this patch. This version keeps just the part of 
Frediano Ziglio's patch that seemed uncontroversial.

 server/reds.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/server/reds.c b/server/reds.c
index 6d625de..992cf03 100644
--- a/server/reds.c
+++ b/server/reds.c
@@ -4507,8 +4507,8 @@ static RedCharDeviceVDIPort 
*red_char_device_vdi_port_new(RedsState *reds)
 {
     return g_object_new(RED_TYPE_CHAR_DEVICE_VDIPORT,
                         "spice-server", reds,
-                        "client-tokens-interval", REDS_TOKENS_TO_SEND,
-                        "self-tokens", REDS_NUM_INTERNAL_AGENT_MESSAGES,
+                        "client-tokens-interval", (guint64)REDS_TOKENS_TO_SEND,
+                        "self-tokens", 
(guint64)REDS_NUM_INTERNAL_AGENT_MESSAGES,
                         "opaque", reds,
                         NULL);
 }
-- 
2.8.1
_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to