From d39f791f6d262a76f1ac8a52eed97c8316cc5e2c Mon Sep 17 00:00:00 2001 From: Abtin Keshavarzian Date: Wed, 15 Jan 2025 13:54:36 -0800 Subject: [PATCH] [border-agent] remove unnecessary state checks in `CoapDtlsSession` methods (#11165) This commit removes unnecessary Border Agent state checks in `CoapDtlsSession` methods. When the Border Agent is stopped, the DTLS transport is closed, and all associated sessions are cleared and removed. --- src/core/meshcop/border_agent.cpp | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/core/meshcop/border_agent.cpp b/src/core/meshcop/border_agent.cpp index 90b5aaec0..014921d4e 100644 --- a/src/core/meshcop/border_agent.cpp +++ b/src/core/meshcop/border_agent.cpp @@ -633,8 +633,6 @@ Error BorderAgent::CoapDtlsSession::ForwardToLeader(const Coap::Message &aMes bool separate = false; OffsetRange offsetRange; - VerifyOrExit(Get().mState != kStateStopped); - switch (aUri) { case kUriLeaderPetition: @@ -877,8 +875,6 @@ void BorderAgent::CoapDtlsSession::HandleTmfProxyTx(Coap::Message &aMessage) OffsetRange offsetRange; UdpEncapsulationTlvHeader udpEncapHeader; - VerifyOrExit(Get().mState != kStateStopped); - SuccessOrExit(error = Tlv::FindTlvValueOffsetRange(aMessage, Tlv::kUdpEncapsulation, offsetRange)); SuccessOrExit(error = aMessage.Read(offsetRange, udpEncapHeader)); @@ -912,8 +908,6 @@ void BorderAgent::CoapDtlsSession::HandleTmfRelayTx(Coap::Message &aMessage) Tmf::MessageInfo messageInfo(GetInstance()); OffsetRange offsetRange; - VerifyOrExit(Get().mState != kStateStopped); - VerifyOrExit(aMessage.IsNonConfirmablePostRequest()); SuccessOrExit(error = Tlv::Find(aMessage, joinerRouterRloc));