Skip to content

Commit 9532b45

Browse files
committed
dynamic size check for each target
1 parent 83d903c commit 9532b45

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

src/PPNet.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@ size_t PPNet::WriteMessage(AnyMessage msg)
7070

7171
// calculate package size (1=code, 4=checksum, rest=payload)
7272
auto totalSize = 1 + 4 + this->packer.size();
73-
assert(totalSize < 255);
7473

7574
auto checksum = PPNetwork::Utils::adler32(packer.data(), packer.size());
7675
uint8_t checksum_arr[4] = {0};
@@ -79,19 +78,22 @@ size_t PPNet::WriteMessage(AnyMessage msg)
7978
switch (this->targetType)
8079
{
8180
case WriteTargetType::RAW:
81+
assert(totalSize < 255);
8282
this->output->write(static_cast<uint8_t>(type));
8383
this->output->write(checksum_arr, 4);
8484
this->output->write(packer.data(), packer.size());
8585
this->output->flush();
8686
break;
8787
case WriteTargetType::SUNTECH:
88+
assert(totalSize < 255);
8889
this->output->write(static_cast<uint8_t>(type));
8990
this->output->write(checksum_arr, 4);
9091
this->output->write(packer.data(), packer.size());
9192
this->output->write("\r\n");
9293
this->output->flush();
9394
break;
9495
case WriteTargetType::AOVX:
96+
assert(totalSize < 511);
9597
{
9698
unsigned long currentTime = millis();
9799
unsigned long elapsedTime = currentTime - this->lastMessageTime;

0 commit comments

Comments
 (0)