[Spice-devel] [PATCH 2/2] server: init all fields on SpiceMsgDisplayStreamCreate

[Christophe Fergeau]

This patch is an RFC

red_display_marshall_stream_start initializes a
SpiceMsgDisplayStreamCreate structure before marshalling it and
sending it on the wire. However, it never fills
SpiceMsgDisplayStreamCreate::stamp which then causes a complaint
from valgrind. Initializing it is easy enough, however I have no idea
if 0 is an acceptable value. I put a semi-random value for now in the
hope that someone can enlighten me as to what I can use for ::stamp.
---
 server/red_worker.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/server/red_worker.c b/server/red_worker.c
index efedc19..dc80259 100644
--- a/server/red_worker.c
+++ b/server/red_worker.c
@@ -7718,6 +7718,8 @@ static void red_display_marshall_stream_start(DisplayChannel *display_channel,
         stream_create.clip.rects = &clip_rects;
     }
 
+    stream_create.stamp = 0xdeadbeef;
+
     spice_marshall_msg_display_stream_create(base_marshaller, &stream_create);
 }
 
-- 
1.7.6