Changeset 8a237c7 for lib/trace_parallel.c

Timestamp:

06/27/18 11:14:47 (3 years ago)

Author:

Shane Alcock <salcock@…>

Branches:

cachetimestamps, develop, master, rc-4.0.4, ringdecrementfix, ringperformance

Children:

3b6ca9a, d228977

Parents:

5577521

Message:

Fix uninitialised memory complaints when sending first packet msg.

Valgrind doesn't like it when we write unpacked structures without
explicitly setting all bytes (including the padding ones) to zero
beforehand.

File:

• lib/trace_parallel.c (modified) (2 diffs)

lib/trace_parallel.c

@@ -1048 +1048 @@
         ASSERT_RET(pthread_spin_unlock(&libtrace->first_packets.lock), == 0);
 
+        memset(&mesg, 0, sizeof(libtrace_message_t));
         mesg.code = MESSAGE_FIRST_PACKET;
         trace_message_reporter(libtrace, &mesg);
@@ -1195 +1196 @@
 
         gettimeofday(&prev, NULL);
+        memset(&message, 0, sizeof(libtrace_message_t));
         message.code = MESSAGE_TICK_INTERVAL;