WiVICommandPacket: Remove length check
parent
9871430288
commit
124bd3b310
|
|
@ -15,9 +15,6 @@ std::shared_ptr<WiVI::ResponseMessage> WiVI::CommandPacket::DecodeToMessage(cons
|
||||||
if(bytestream.size() < sizeof(WiVI::CommandPacket::Result))
|
if(bytestream.size() < sizeof(WiVI::CommandPacket::Result))
|
||||||
return {};
|
return {};
|
||||||
|
|
||||||
if(bytestream.size() != sizeof(WiVI::CommandPacket::Header) + header.length)
|
|
||||||
return {};
|
|
||||||
|
|
||||||
const auto& decoded = *reinterpret_cast<const WiVI::CommandPacket::Result*>(bytestream.data());
|
const auto& decoded = *reinterpret_cast<const WiVI::CommandPacket::Result*>(bytestream.data());
|
||||||
msg->responseTo = decoded.responseTo;
|
msg->responseTo = decoded.responseTo;
|
||||||
msg->success = decoded.result != 0;
|
msg->success = decoded.result != 0;
|
||||||
|
|
@ -28,9 +25,6 @@ std::shared_ptr<WiVI::ResponseMessage> WiVI::CommandPacket::DecodeToMessage(cons
|
||||||
if(bytestream.size() < sizeof(WiVI::CommandPacket::SetSignal))
|
if(bytestream.size() < sizeof(WiVI::CommandPacket::SetSignal))
|
||||||
return {};
|
return {};
|
||||||
|
|
||||||
if(bytestream.size() != sizeof(WiVI::CommandPacket::SetSignal) + header.length)
|
|
||||||
return {};
|
|
||||||
|
|
||||||
const auto& setSignal = *reinterpret_cast<const WiVI::CommandPacket::SetSignal*>(bytestream.data());
|
const auto& setSignal = *reinterpret_cast<const WiVI::CommandPacket::SetSignal*>(bytestream.data());
|
||||||
msg->responseTo = WiVI::Command::GetSignal;
|
msg->responseTo = WiVI::Command::GetSignal;
|
||||||
msg->value = setSignal.value.ValueInt32;
|
msg->value = setSignal.value.ValueInt32;
|
||||||
|
|
@ -40,9 +34,6 @@ std::shared_ptr<WiVI::ResponseMessage> WiVI::CommandPacket::DecodeToMessage(cons
|
||||||
if(bytestream.size() < sizeof(WiVI::CommandPacket::GetAll))
|
if(bytestream.size() < sizeof(WiVI::CommandPacket::GetAll))
|
||||||
return {};
|
return {};
|
||||||
|
|
||||||
if(bytestream.size() != sizeof(WiVI::CommandPacket::GetAll) + header.length)
|
|
||||||
return {};
|
|
||||||
|
|
||||||
const auto& getAll = *reinterpret_cast<const WiVI::CommandPacket::GetAll*>(bytestream.data());
|
const auto& getAll = *reinterpret_cast<const WiVI::CommandPacket::GetAll*>(bytestream.data());
|
||||||
msg->responseTo = WiVI::Command::GetAll;
|
msg->responseTo = WiVI::Command::GetAll;
|
||||||
msg->info.emplace();
|
msg->info.emplace();
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue