Skip to content

B
brpc
Commit c9e3131a authored by Ge Jun's avatar Ge Jun
Browse files
Options

the buffer passed to TMemoryBuffer should be malloc-ed instead of new[]

parent 18a96d5b
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 2 deletions
src/brpc/policy/thrift_protocol.cpp
View file @ c9e3131a
...@@ -122,7 +122,7 @@ bool ReadThriftStruct(const butil::IOBuf& body, ...@@ -122,7 +122,7 @@ bool ReadThriftStruct(const butil::IOBuf& body,
ThriftMessageBase* raw_msg, ThriftMessageBase* raw_msg,
int16_t expected_fid) { int16_t expected_fid) {
const size_t body_len = body.size(); const size_t body_len = body.size();
uint8_t* thrift_buffer = new uint8_t[body_len]; uint8_t* thrift_buffer = (uint8_t*)malloc(body_len);
body.copy_to(thrift_buffer, body_len); body.copy_to(thrift_buffer, body_len);
auto in_buffer = auto in_buffer =
THRIFT_STDCXX::make_shared<apache::thrift::transport::TMemoryBuffer>( THRIFT_STDCXX::make_shared<apache::thrift::transport::TMemoryBuffer>(
...@@ -165,7 +165,7 @@ bool ReadThriftStruct(const butil::IOBuf& body, ...@@ -165,7 +165,7 @@ bool ReadThriftStruct(const butil::IOBuf& body,
void ReadThriftException(const butil::IOBuf& body, void ReadThriftException(const butil::IOBuf& body,
::apache::thrift::TApplicationException* x) { ::apache::thrift::TApplicationException* x) {
size_t body_len = body.size(); size_t body_len = body.size();
uint8_t* thrift_buffer = new uint8_t[body_len]; uint8_t* thrift_buffer = (uint8_t*)malloc(body_len);
body.copy_to(thrift_buffer, body_len); body.copy_to(thrift_buffer, body_len);
auto in_buffer = auto in_buffer =
THRIFT_STDCXX::make_shared<apache::thrift::transport::TMemoryBuffer>( THRIFT_STDCXX::make_shared<apache::thrift::transport::TMemoryBuffer>(
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment