aboutsummaryrefslogtreecommitdiff
path: root/src/zenserver-test/zenserver-test.cpp
diff options
context:
space:
mode:
authorDan Engelbrecht <[email protected]>2024-04-04 13:37:17 +0200
committerGitHub Enterprise <[email protected]>2024-04-04 13:37:17 +0200
commit7d2421dd0aaebf75a2cbe5d6a0827f1d064c0c3f (patch)
tree5b65f6c8f4c2986041805541f68d17072e3b8d37 /src/zenserver-test/zenserver-test.cpp
parentvalidate rpc chunk responses (#36) (diff)
downloadzen-7d2421dd0aaebf75a2cbe5d6a0827f1d064c0c3f.tar.xz
zen-7d2421dd0aaebf75a2cbe5d6a0827f1d064c0c3f.zip
hardening parsepackagemessage (#38)
* hardening of ParsePackageMessage and extended details when malformed attachments are detected
Diffstat (limited to 'src/zenserver-test/zenserver-test.cpp')
-rw-r--r--src/zenserver-test/zenserver-test.cpp10
1 files changed, 4 insertions, 6 deletions
diff --git a/src/zenserver-test/zenserver-test.cpp b/src/zenserver-test/zenserver-test.cpp
index 412a2d26a..325b15e3f 100644
--- a/src/zenserver-test/zenserver-test.cpp
+++ b/src/zenserver-test/zenserver-test.cpp
@@ -1228,12 +1228,10 @@ TEST_CASE("zcache.rpc")
if (Result.status_code == 200)
{
CbPackage Response = ParsePackageMessage(zen::IoBuffer(zen::IoBuffer::Wrap, Result.text.data(), Result.text.size()));
- if (!Response.IsNull())
- {
- OutResult.Response = std::move(Response);
- CHECK(OutResult.Result.Parse(OutResult.Response));
- OutResult.Success = true;
- }
+ CHECK(!Response.IsNull());
+ OutResult.Response = std::move(Response);
+ CHECK(OutResult.Result.Parse(OutResult.Response));
+ OutResult.Success = true;
}
return OutResult;