diff options
| author | Kevin Rocard <kevinx.rocard@intel.com> | 2014-12-19 15:11:21 +0100 |
|---|---|---|
| committer | Eric Laurent <elaurent@google.com> | 2015-04-24 13:32:47 -0700 |
| commit | 0a7b18ef0f72b4a90b663c0564de632a70d6bc5f (patch) | |
| tree | e2b61855415d42180b1431de232b4be88928c131 | |
| parent | 4e69cce75acbdaf9cf2933c229908dc910f29a9f (diff) | |
| download | external_parameter-framework-0a7b18ef0f72b4a90b663c0564de632a70d6bc5f.zip external_parameter-framework-0a7b18ef0f72b4a90b663c0564de632a70d6bc5f.tar.gz external_parameter-framework-0a7b18ef0f72b4a90b663c0564de632a70d6bc5f.tar.bz2 | |
[remote-processor] Remove useless new
Message::readString was temporally allocating a buffer
on the heap (with new).
Use variable length arrays to allocate the buffer on the stack.
Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
| -rw-r--r-- | remote-processor/Message.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/remote-processor/Message.cpp b/remote-processor/Message.cpp index ee3652d..8521117 100644 --- a/remote-processor/Message.cpp +++ b/remote-processor/Message.cpp @@ -98,7 +98,7 @@ void CMessage::readString(string& strData) readData(&uiSize, sizeof(uiSize)); // Data - char* pcData = new char[uiSize + 1]; + char pcData[uiSize + 1]; // Content readData(pcData, uiSize); @@ -108,9 +108,6 @@ void CMessage::readString(string& strData) // Output strData = pcData; - - // Delete - delete [] pcData; } uint32_t CMessage::getStringSize(const string& strData) const |