From 76a0334090a460648ee4ccf5e6b538084435b2d0 Mon Sep 17 00:00:00 2001
From: Abtin Keshavarzian <abtink@google.com>
Date: Wed, 20 Nov 2024 10:09:01 -0800
Subject: [PATCH] [secure-transport] remove extra `mbedtls_ssl_close_notify()`
 calls (#10947)

This commit removes the calls to `mbedtls_ssl_close_notify()` in
`SecureTransport::Process()` when, based on `rval`, it is determined
that `Disconnect()` should be called, as the `Disconnect()` method
itself will call `mbedtls_ssl_close_notify()`.
---
 src/core/meshcop/secure_transport.cpp | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/core/meshcop/secure_transport.cpp b/src/core/meshcop/secure_transport.cpp
index aa2c69c6f..27fa1e923 100644
--- a/src/core/meshcop/secure_transport.cpp
+++ b/src/core/meshcop/secure_transport.cpp
@@ -1049,7 +1049,6 @@ void SecureTransport::Process(void)
             break;
 
         case MBEDTLS_ERR_SSL_PEER_CLOSE_NOTIFY:
-            mbedtls_ssl_close_notify(&mSsl);
             disconnectEvent = kDisconnectedPeerClosed;
             break;
 
@@ -1057,7 +1056,6 @@ void SecureTransport::Process(void)
             break;
 
         case MBEDTLS_ERR_SSL_FATAL_ALERT_MESSAGE:
-            mbedtls_ssl_close_notify(&mSsl);
             disconnectEvent = kDisconnectedError;
             break;