diff options
| author | Scott Powell <sqij@protonmail.com> | 2026-02-26 15:36:21 +1100 |
|---|---|---|
| committer | Scott Powell <sqij@protonmail.com> | 2026-02-26 15:36:21 +1100 |
| commit | b67decfba06bbcafbb6909157b46a37c09106930 (patch) | |
| tree | 25365e32c5c6bc7b4a2ff2ae7f595d4fad67864d /src/Packet.cpp | |
| parent | ca81f645ed0f69a8bd5395707e77124e66fddebb (diff) | |
* bug fix: Packet::writeTo(), Packet::readFrom()
Diffstat (limited to 'src/Packet.cpp')
| -rw-r--r-- | src/Packet.cpp | 9 |
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 |
