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

Error checking for Encoder::encode UpdateLEDState

checksum-failure-logging
Paul Hollinsky 2019-07-23 15:04:50 -04:00
parent 77eed8f58d
commit 21c3fead4a
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
communication/encoder.cpp
View File

@ -106,14 +106,18 @@ bool Encoder::encode(std::vector<uint8_t>& result, Command cmd, std::vector<uint
if(cmd == Command::UpdateLEDState) { if(cmd == Command::UpdateLEDState) {
/* NetID::Device is a super old command type. /* NetID::Device is a super old command type.
* It has a leading 0x00 byte, a byte for command, and a byte for an argument. * It has a leading 0x00 byte, a byte for command, and a byte for an argument.
* In this case, command 0x06 is SetLEDState. * In this case, command 0x06 is SetLEDState.
* This old command type is not really used anywhere else. * This old command type is not really used anywhere else.
*/ */
if (arguments.empty()) {
report(APIEvent::Type::MessageFormattingError, APIEvent::Severity::Error);
return false;
}
msg->network = Network::NetID::Device; msg->network = Network::NetID::Device;
msg->data.reserve(3); msg->data.reserve(3);
msg->data.push_back(0x00); msg->data.push_back(0x00);
msg->data.push_back(0x06); msg->data.push_back(0x06); // SetLEDState
msg->data.push_back(arguments.at(0)); msg->data.push_back(arguments.at(0)); // See Device::LEDState
} else { } else {
msg->network = Network::NetID::Main51; msg->network = Network::NetID::Main51;
msg->data.reserve(arguments.size() + 1); msg->data.reserve(arguments.size() + 1);