From 70a06f3cd65915b9262723e0883cfc100672bf61 Mon Sep 17 00:00:00 2001 From: Andy CA6JAU Date: Sun, 29 Jul 2018 19:41:19 -0400 Subject: [PATCH] Fix command length for dump function in WiresX --- YSFGateway/WiresX.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/YSFGateway/WiresX.cpp b/YSFGateway/WiresX.cpp index 1489d2f..8013f7a 100644 --- a/YSFGateway/WiresX.cpp +++ b/YSFGateway/WiresX.cpp @@ -213,7 +213,8 @@ WX_STATUS CWiresX::process(const unsigned char* data, const unsigned char* sourc bool valid = false; // Find the end marker - for (unsigned int i = (fn - 1U) * 40U + 20U; i > 0U; i--) { + unsigned int cmd_len = (fn - 1U) * 40U + 20U; + for (unsigned int i = cmd_len; i > 0U; i--) { if (m_command[i] == 0x03U) { unsigned char crc = CCRC::addCRC(m_command, i + 1U); if (crc == m_command[i + 1U]) @@ -237,7 +238,7 @@ WX_STATUS CWiresX::process(const unsigned char* data, const unsigned char* sourc processDisconnect(source); return WXS_DISCONNECT; } else { - CUtils::dump("Unknown Wires-X command", m_command, fn * 20U); + CUtils::dump("Unknown Wires-X command", m_command, cmd_len); return WXS_NONE; } }