[coap] add NULL check in Message::Clone() (#3469)

This commit is contained in:
Jonathan Hui
2019-01-17 07:33:02 -08:00
committed by GitHub
parent e45107dac7
commit b25e0bb4c8
2 changed files with 13 additions and 8 deletions
+12
View File
@@ -365,6 +365,18 @@ void Message::SetDefaultResponseHeader(const Message &aRequest)
SetToken(aRequest.GetToken(), aRequest.GetTokenLength());
}
Message *Message::Clone(uint16_t aLength) const
{
Message *message = static_cast<Message *>(ot::Message::Clone(aLength));
VerifyOrExit(message != NULL);
memcpy(&message->GetHelpData(), &GetHelpData(), sizeof(GetHelpData()));
exit:
return message;
}
#if OPENTHREAD_ENABLE_APPLICATION_COAP
const char *Message::CodeToString(void) const
{
+1 -8
View File
@@ -491,14 +491,7 @@ public:
* @returns A pointer to the message or NULL if insufficient message buffers are available.
*
*/
Message *Clone(uint16_t aLength) const
{
Message *message = static_cast<Message *>(ot::Message::Clone(aLength));
memcpy(&message->GetHelpData(), &GetHelpData(), sizeof(GetHelpData()));
return message;
}
Message *Clone(uint16_t aLength) const;
/**
* This method creates a copy of the message.