From 91e1d9cfa329129d5c35d2fd84bcf10c99f35710 Mon Sep 17 00:00:00 2001 From: canisLupus1313 <85344032+canisLupus1313@users.noreply.github.com> Date: Wed, 16 Feb 2022 19:57:37 +0100 Subject: [PATCH] [ip6] fix static code analysis errors (#7422) This commit fixes issues found during static code analysis. --- src/core/meshcop/dataset_manager.cpp | 2 +- src/core/net/ip6.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/core/meshcop/dataset_manager.cpp b/src/core/meshcop/dataset_manager.cpp index d241b1f1d..4bb845c8b 100644 --- a/src/core/meshcop/dataset_manager.cpp +++ b/src/core/meshcop/dataset_manager.cpp @@ -372,7 +372,7 @@ void DatasetManager::HandleGet(const Coap::Message &aMessage, const Ip6::Message while (offset < aMessage.GetLength()) { - IgnoreError(aMessage.Read(offset, tlv)); + SuccessOrExit(aMessage.Read(offset, tlv)); if (tlv.GetType() == Tlv::kGet) { diff --git a/src/core/net/ip6.cpp b/src/core/net/ip6.cpp index 70231aa1d..9ccbc1d73 100644 --- a/src/core/net/ip6.cpp +++ b/src/core/net/ip6.cpp @@ -262,7 +262,7 @@ Error Ip6::InsertMplOption(Message &aMessage, Header &aHeader, MessageInfo &aMes OptionMpl mplOption; // read existing hop-by-hop option header - IgnoreError(aMessage.Read(0, hbh)); + SuccessOrExit(error = aMessage.Read(0, hbh)); hbhLength = (hbh.GetLength() + 1) * 8; VerifyOrExit(hbhLength <= aHeader.GetPayloadLength(), error = kErrorParse);