From fea6b5a87c4e8e94c1a060669fd3c0b57d08956f Mon Sep 17 00:00:00 2001 From: Daniel Caujolle-Bert Date: Fri, 3 Dec 2021 06:36:33 +0100 Subject: [PATCH] Fix buffer length checking on MMDVMController::getResponse(): many frame length are 114 bytes long, limit is hardcoded to 100, hence use BUFFER_LENGTH instead (200). --- Common/MMDVMController.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Common/MMDVMController.cpp b/Common/MMDVMController.cpp index 32e0e54..c101c96 100644 --- a/Common/MMDVMController.cpp +++ b/Common/MMDVMController.cpp @@ -570,7 +570,7 @@ RESP_TYPE_MMDVM CMMDVMController::getResponse(unsigned char *buffer, unsigned in length = buffer[1U]; - if (length >= 100U) { + if (length >= BUFFER_LENGTH) { wxLogError(wxT("Invalid data received from the MMDVM")); CUtils::dump(wxT("Data"), buffer, 2U); return RTDVM_TIMEOUT;