From 8c92497a7bfce38370b62030ab0635fe56bbb10f Mon Sep 17 00:00:00 2001 From: Jonathan Hui Date: Thu, 16 May 2024 18:19:34 -0700 Subject: [PATCH] [secure-transport] remove redundant null check (#10254) --- src/core/meshcop/secure_transport.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/core/meshcop/secure_transport.cpp b/src/core/meshcop/secure_transport.cpp index 73b0e9a5b..27ed62988 100644 --- a/src/core/meshcop/secure_transport.cpp +++ b/src/core/meshcop/secure_transport.cpp @@ -638,15 +638,13 @@ Error SecureTransport::GetPeerSubjectAttributeByOid(const char *aOid, VerifyOrExit(aAttributeBuffer != nullptr, error = kErrorNoBufs); VerifyOrExit(peerCert != nullptr, error = kErrorInvalidState); + data = mbedtls_asn1_find_named_data(&peerCert->subject, aOid, aOidLength); VerifyOrExit(data != nullptr, error = kErrorNotFound); + length = data->val.len; VerifyOrExit(length <= attributeBufferSize, error = kErrorNoBufs); - - if (aAttributeLength != nullptr) - { - *aAttributeLength = length; - } + *aAttributeLength = length; if (aAsn1Type != nullptr) {