commit 6f82926ea4f9fb1ede66b8c67e911a1c1c66aeb3
author Blake Bender <bbender@pivotal.io> Thu Mar 04 14:24:18 2021 -0800
committer GitHub <noreply@github.com> Thu Mar 04 14:24:18 2021 -0800
tree 5437b4b5cffef69b086bb03cbfb433d7d56e89fc
parent 38fa9eef45da2a0743410d24c5852e8dd3a69494

GEODE-8994: Fix log line in EventId ctor (#756)

- memId isn't a string, need to convert it first or we'll log garbage.

cppcache/src/EventId.cpp

diff --git a/cppcache/src/EventId.cpp b/cppcache/src/EventId.cpp
index 62c0539..d53090a 100644
--- a/cppcache/src/EventId.cpp
+++ b/cppcache/src/EventId.cpp
@@ -23,6 +23,7 @@
 #include <geode/DataInput.hpp>
 
 #include "ClientProxyMembershipID.hpp"
+#include "Utils.hpp"
 #include "util/Log.hpp"
 
 namespace apache {
@@ -137,7 +138,8 @@
 EventId::EventId(char* memId, uint32_t memIdLen, int64_t thr, int64_t seq) {
   LOGDEBUG("EventId::EventId(%p) - memId=%s, memIdLen=%d, thr=%" PRId64
            ", seq=%" PRId64,
-           this, memId, memIdLen, thr, seq);
+           this, Utils::convertBytesToString(memId, memIdLen).c_str(), memIdLen,
+           thr, seq);
   // TODO: statics being assigned; not thread-safe??
   std::memcpy(clientId_, memId, memIdLen);
   clientIdLength_ = memIdLen;