diff options
author | Jan Tattermusch <jtattermusch@users.noreply.github.com> | 2018-11-08 16:48:26 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-11-08 16:48:26 +0100 |
commit | 4e1610bd41f2e2b563c1a325d6769dbdd2163022 (patch) | |
tree | aa8500abdd8bb89f85c895109c885a9e9ca920df /test/cpp | |
parent | b3ab16b465833f4843036a7a0c4072600dd1a65a (diff) | |
parent | 432b34bf7430fd01f7dd3b4e7ba262846bbe1b7c (diff) |
Merge pull request #17140 from vjpai/qps
Initialize values in memory before creating slice
Diffstat (limited to 'test/cpp')
-rw-r--r-- | test/cpp/qps/client.h | 8 | ||||
-rw-r--r-- | test/cpp/qps/server_async.cc | 1 |
2 files changed, 6 insertions, 3 deletions
diff --git a/test/cpp/qps/client.h b/test/cpp/qps/client.h index 4ed34e0405..668d941916 100644 --- a/test/cpp/qps/client.h +++ b/test/cpp/qps/client.h @@ -94,9 +94,11 @@ class ClientRequestCreator<ByteBuffer> { public: ClientRequestCreator(ByteBuffer* req, const PayloadConfig& payload_config) { if (payload_config.has_bytebuf_params()) { - std::unique_ptr<char[]> buf( - new char[payload_config.bytebuf_params().req_size()]); - Slice slice(buf.get(), payload_config.bytebuf_params().req_size()); + size_t req_sz = + static_cast<size_t>(payload_config.bytebuf_params().req_size()); + std::unique_ptr<char[]> buf(new char[req_sz]); + memset(buf.get(), 0, req_sz); + Slice slice(buf.get(), req_sz); *req = ByteBuffer(&slice, 1); } else { GPR_ASSERT(false); // not appropriate for this specialization diff --git a/test/cpp/qps/server_async.cc b/test/cpp/qps/server_async.cc index 5cd975cf74..a5f8347c26 100644 --- a/test/cpp/qps/server_async.cc +++ b/test/cpp/qps/server_async.cc @@ -562,6 +562,7 @@ static Status ProcessGenericRPC(const PayloadConfig& payload_config, request->Clear(); int resp_size = payload_config.bytebuf_params().resp_size(); std::unique_ptr<char[]> buf(new char[resp_size]); + memset(buf.get(), 0, static_cast<size_t>(resp_size)); Slice slice(buf.get(), resp_size); *response = ByteBuffer(&slice, 1); return Status::OK; |