aboutsummaryrefslogtreecommitdiff
path: root/src/Packet.cpp
diff options
context:
space:
mode:
authorScott Powell <sqij@protonmail.com>2026-02-26 15:36:21 +1100
committerScott Powell <sqij@protonmail.com>2026-02-26 15:36:21 +1100
commitb67decfba06bbcafbb6909157b46a37c09106930 (patch)
tree25365e32c5c6bc7b4a2ff2ae7f595d4fad67864d /src/Packet.cpp
parentca81f645ed0f69a8bd5395707e77124e66fddebb (diff)
* bug fix: Packet::writeTo(), Packet::readFrom()
Diffstat (limited to 'src/Packet.cpp')
-rw-r--r--src/Packet.cpp9
1 files changed, 6 insertions, 3 deletions
diff --git a/src/Packet.cpp b/src/Packet.cpp
index 934020b3..aad3e2f4 100644
--- a/src/Packet.cpp
+++ b/src/Packet.cpp
@@ -57,7 +57,7 @@ uint8_t Packet::writeTo(uint8_t dest[]) const {
memcpy(&dest[i], &transport_codes[1], 2); i += 2;
}
dest[i++] = path_len;
- memcpy(&dest[i], path, path_len); i += path_len;
+ i += writePath(&dest[i], path, path_len);
memcpy(&dest[i], payload, payload_len); i += payload_len;
return i;
}
@@ -72,8 +72,11 @@ bool Packet::readFrom(const uint8_t src[], uint8_t len) {
transport_codes[0] = transport_codes[1] = 0;
}
path_len = src[i++];
- if (path_len > sizeof(path)) return false; // bad encoding
- memcpy(path, &src[i], path_len); i += path_len;
+ if (!isValidPathLen(path_len)) return false; // bad encoding
+
+ uint8_t bl = getPathByteLen();
+ memcpy(path, &src[i], bl); i += bl;
+
if (i >= len) return false; // bad encoding
payload_len = len - i;
if (payload_len > sizeof(payload)) return false; // bad encoding