diff options
| author | Dan Engelbrecht <[email protected]> | 2025-09-26 10:26:34 +0200 |
|---|---|---|
| committer | GitHub Enterprise <[email protected]> | 2025-09-26 10:26:34 +0200 |
| commit | 347153218dd09e3806e5b27eb51f538768f27035 (patch) | |
| tree | 81ef102cc116ed2c751e1af91d83db32ffa8d369 /src/zencore/compactbinaryutil.cpp | |
| parent | Merge pull request #509 from ue-foundation/zs/put-overwrite-policy-response (diff) | |
| download | zen-347153218dd09e3806e5b27eb51f538768f27035.tar.xz zen-347153218dd09e3806e5b27eb51f538768f27035.zip | |
new append op rpc method (#511)
- Feature: New `/prj/{project}/{oplog}/rpc` endpoint method `appendops` to send an array of oplog ops and receiving a list of `need` for attachments not present
- Feature: Added `usingtmpfiles` boolean field to `/prj/{project}/{oplog}/rpc` method `putchunks` to be explicit about allowing move of temp attachment files
- Improvement: Added additional validation of compact binary objects when reading from disk/receiving from client
- Improvement: Windows: Added fallback code to use standard `MoveFile` api when rename via `SetFileInformationByHandle` fails in `MoveToFile` (used by filecas)
Diffstat (limited to 'src/zencore/compactbinaryutil.cpp')
| -rw-r--r-- | src/zencore/compactbinaryutil.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/zencore/compactbinaryutil.cpp b/src/zencore/compactbinaryutil.cpp index c8cde21c3..074bdaffd 100644 --- a/src/zencore/compactbinaryutil.cpp +++ b/src/zencore/compactbinaryutil.cpp @@ -14,7 +14,13 @@ ValidateAndReadCompactBinaryObject(const SharedBuffer&& Payload, CbValidateError { if (OutError = ValidateCompactBinary(Payload.GetView(), CbValidateMode::Default); OutError == CbValidateError::None) { - return CbObject(std::move(Payload)); + CbObject Object(std::move(Payload)); + if (Object.GetView().GetSize() != Payload.GetSize()) + { + OutError |= CbValidateError::OutOfBounds; + return {}; + } + return Object; } } return CbObject(); |