might be an important fix
This commit is contained in:
parent
467ee0028a
commit
a35b28fc57
@ -16,6 +16,11 @@ will consitute of a breaking change warranting a new major release:
|
|||||||
|
|
||||||
# [unreleased]
|
# [unreleased]
|
||||||
|
|
||||||
|
## Fixed
|
||||||
|
|
||||||
|
- Added null termination for PLOC MPSoC image taking command which could possibly lead to
|
||||||
|
default target filenames.
|
||||||
|
|
||||||
# [v8.1.0] 2024-05-29
|
# [v8.1.0] 2024-05-29
|
||||||
|
|
||||||
## Fixed
|
## Fixed
|
||||||
|
@ -920,13 +920,14 @@ class TcCamTakePic : public TcBase {
|
|||||||
}
|
}
|
||||||
size_t deserLen = commandDataLen;
|
size_t deserLen = commandDataLen;
|
||||||
size_t serLen = 0;
|
size_t serLen = 0;
|
||||||
fileName = reinterpret_cast<const char*>(commandData);
|
fileName = std::string(reinterpret_cast<const char*>(commandData));
|
||||||
if (fileName.size() > MAX_FILENAME_SIZE) {
|
if (fileName.size() > MAX_FILENAME_SIZE) {
|
||||||
return FILENAME_TOO_LONG;
|
return FILENAME_TOO_LONG;
|
||||||
}
|
}
|
||||||
deserLen -= fileName.length() + 1;
|
deserLen -= fileName.length() + 1;
|
||||||
*dataPtr += fileName.length() + 1;
|
*dataPtr += fileName.length() + 1;
|
||||||
uint8_t** payloadPtr = &payloadStart;
|
uint8_t** payloadPtr = &payloadStart;
|
||||||
|
memset(payloadStart, 0, FILENAME_FIELD_SIZE);
|
||||||
memcpy(payloadStart, fileName.data(), fileName.size());
|
memcpy(payloadStart, fileName.data(), fileName.size());
|
||||||
*payloadPtr += FILENAME_FIELD_SIZE;
|
*payloadPtr += FILENAME_FIELD_SIZE;
|
||||||
serLen += FILENAME_FIELD_SIZE;
|
serLen += FILENAME_FIELD_SIZE;
|
||||||
|
Loading…
x
Reference in New Issue
Block a user