From 8b51bacfd63ffcffa06e2815dd5ee72a45b5de79 Mon Sep 17 00:00:00 2001
From: Nirbhay Choubey <nirbhay@mariadb.com>
Date: Tue, 20 Sep 2016 21:32:53 -0400
Subject: [PATCH] MDEV-10735: Valgrind warnings around Galera SST

While copying the received state Id (uuid:seqno) to an uninitialized
buffer, it was not properly null-terminated.
---
 sql/wsrep_sst.cc | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/sql/wsrep_sst.cc b/sql/wsrep_sst.cc
index d88263d75cb..7c366435472 100644
--- a/sql/wsrep_sst.cc
+++ b/sql/wsrep_sst.cc
@@ -476,13 +476,11 @@ static void* sst_joiner_thread (void* a)
 
       } else {
         // Scan state ID first followed by wsrep_gtid_domain_id.
-        char uuid[512];
         unsigned long int domain_id;
-        size_t len= pos - out + 1;
 
-        if (len > sizeof(uuid)) goto err;       // safety check
-        memcpy(uuid, out, len);                 // including '\0'
-        err= sst_scan_uuid_seqno (uuid, &ret_uuid, &ret_seqno);
+        // Null-terminate the state-id.
+        out[pos - out]= 0;
+        err= sst_scan_uuid_seqno (out, &ret_uuid, &ret_seqno);
 
         if (err)
         {