sven
/
libicsneo
mirror of https://github.com/intrepidcs/libicsneo.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

CANPacket: Fix invalid length check 

This case is already checked above, and further
was invalid since the length had already been
translated to the numeric (0-64) value.
v0.3.0-dev
Paul Hollinsky 2021-05-25 17:23:41 -04:00
parent ad07af160c
commit 76b6ecbf04
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
communication/packet/canpacket.cpp
View File

@ -124,7 +124,7 @@ bool HardwareCANPacket::EncodeFromMessage(const CANMessage& message, std::vector
const optional<uint8_t> lenFromDLC = CANFD_DLCToLength(message.dlcOnWire); const optional<uint8_t> lenFromDLC = CANFD_DLCToLength(message.dlcOnWire);
if (lenFromDLC.has_value() && *lenFromDLC != 0) { if (lenFromDLC.has_value() && *lenFromDLC != 0) {
if (*lenFromDLC < lengthNibble || *lenFromDLC > 0xF) { if (*lenFromDLC < lengthNibble) {
report(APIEvent::Type::MessageMaxLengthExceeded, APIEvent::Severity::Error); report(APIEvent::Type::MessageMaxLengthExceeded, APIEvent::Severity::Error);
return false; return false;
} }