Blake Bender created GEODE-8994:
-----------------------------------

             Summary: Log line for EventId ctor potentially contains garbage 
string
                 Key: GEODE-8994
                 URL: https://issues.apache.org/jira/browse/GEODE-8994
             Project: Geode
          Issue Type: Bug
          Components: native client
            Reporter: Blake Bender


The following logging was recently added to geode-native for debugging:

```

LOGDEBUG("EventId::EventId(%p) - memId=%s, memIdLen=%d, thr=%" PRId64
 ", seq=%" PRId64, this, memId, memIdLen, thr, seq);

```

The variable `memId` in this case is of type `char*` but is NOT a valid string, 
rather just a byte buffer.  Logging it this way can potentially print out 
garbage, causing errors in parsing tools like gnmsg or other apps when 
attempting to read it.  `memId` needs to be decoded into a string 
representation of the bytes, and that logged, rather than attempting to print 
it out raw.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to