From 913a0e6c456c2cbcec5359317ab56519c3e47d01 Mon Sep 17 00:00:00 2001
From: Song Guo <songguo@google.com>
Date: Tue, 12 Jan 2021 11:50:17 +0800
Subject: [PATCH] Fix compile warning and typo in WDMClient

---
 src/device-manager/WeaveDataManagementClient.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/device-manager/WeaveDataManagementClient.cpp b/src/device-manager/WeaveDataManagementClient.cpp
index 6fb3d927c6..e47e672a12 100644
--- a/src/device-manager/WeaveDataManagementClient.cpp
+++ b/src/device-manager/WeaveDataManagementClient.cpp
@@ -937,7 +937,7 @@ void WriteEscapedString(const char * apStr, size_t aLen, std::string & aBuf)
     // According to UTF8 encoding, all bytes from a multiple byte UTF8 sequence
     // will have 1 as most siginificant bit. So this function will output the
     // multi-byte characters without escape.
-    constexpr char * hex = "0123456789abcdef";
+    constexpr char hex[] = "0123456789abcdef";
     for (size_t i = 0; i < aLen && apStr[i]; i++)
     {
         switch (apStr[i])
@@ -1065,7 +1065,7 @@ WEAVE_ERROR FormatEventData(TLVReader aInReader, std::string & aBuf)
     if (aInReader.GetType() == nl::Weave::TLV::kTLVType_Structure || aInReader.GetType() == nl::Weave::TLV::kTLVType_Array)
     {
         bool insideStructure = (aInReader.GetType() == nl::Weave::TLV::kTLVType_Structure);
-        aBuf += (insideStructure ? ' {' : '[');
+        aBuf += (insideStructure ? '{' : '[');
         const char terminating_char = (insideStructure ? '}' : ']');
 
         nl::Weave::TLV::TLVType type;