From b5e17e07eeded88829aa713ce77e209bca88c881 Mon Sep 17 00:00:00 2001 From: Yakun Xu Date: Wed, 12 Aug 2020 01:22:57 +0800 Subject: [PATCH] [mle] log handling parent request failures (#5355) This commit adds a few logs when handling MLE Parent Request, so that when a child failed to receive a response, the logs can tell what happened. --- src/core/thread/mle_router.cpp | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/core/thread/mle_router.cpp b/src/core/thread/mle_router.cpp index bf351a7da..70c871420 100644 --- a/src/core/thread/mle_router.cpp +++ b/src/core/thread/mle_router.cpp @@ -1675,7 +1675,7 @@ void MleRouter::HandleParentRequest(const Message &aMessage, const Ip6::MessageI if (child == nullptr) { - VerifyOrExit((child = mChildTable.GetNewChild()) != nullptr, OT_NOOP); + VerifyOrExit((child = mChildTable.GetNewChild()) != nullptr, error = OT_ERROR_NO_BUFS); // MAC Address child->SetExtAddress(extAddr); @@ -1996,9 +1996,14 @@ void MleRouter::SendParentResponse(Child *aChild, const Challenge &aChallenge, b exit: - if (error != OT_ERROR_NONE && message != nullptr) + if (error != OT_ERROR_NONE) { - message->Free(); + otLogWarnMle("Failed to send Parent Response: %s", otThreadErrorToString(error)); + + if (message != nullptr) + { + message->Free(); + } } }