Skip to content

Commit

Permalink
Fix nasa#8, Remove unnecessary characters from memory dump event
Browse files Browse the repository at this point in the history
  • Loading branch information
thnkslprpt committed Mar 12, 2023
1 parent b824250 commit d96f62b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion fsw/src/mm_dump.c
Original file line number Diff line number Diff line change
Expand Up @@ -526,7 +526,7 @@ bool MM_DumpInEventCmd(const CFE_SB_Buffer_t *BufPtr)
BytePtr = (uint8 *)DumpBuffer;
for (i = 0; i < CmdPtr->NumOfBytes; i++)
{
snprintf(TempString, MM_DUMPINEVENT_TEMP_CHARS, "0x%02X ", *BytePtr);
snprintf(TempString, MM_DUMPINEVENT_TEMP_CHARS, "%02X", *BytePtr);
strncat(EventString, TempString, MM_DUMPINEVENT_TEMP_CHARS);
BytePtr++;
}
Expand Down
4 changes: 2 additions & 2 deletions fsw/src/mm_dump.h
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,10 @@
*
* The event message format is:
* Message head "Memory Dump: " 13 characters
* Message body "0xFF " 5 characters per dump byte
* Message body "FF" 2 characters per dump byte
* Message tail "from address: 0xFFFFFFFF" 33 characters including NUL on 64-bit system
*/
#define MM_MAX_DUMP_INEVENT_BYTES ((CFE_MISSION_EVS_MAX_MESSAGE_LENGTH - (13 + 33)) / 5)
#define MM_MAX_DUMP_INEVENT_BYTES ((CFE_MISSION_EVS_MAX_MESSAGE_LENGTH - (13 + 33)) / 2)

/**
* \brief Dump in an event scratch string size
Expand Down

0 comments on commit d96f62b

Please sign in to comment.