Skip to content

Commit d55c0da

Browse files
authored
Merge pull request #8613 from SeppoTakalo/tls_fix
Make sure that TLSSocketWrapper::close() is called before the transport is destroyed.
2 parents 909c11b + f92108c commit d55c0da

File tree

2 files changed

+13
-0
lines changed

2 files changed

+13
-0
lines changed

features/netsocket/TLSSocket.cpp

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,4 +35,13 @@ nsapi_error_t TLSSocket::connect(const char *host, uint16_t port)
3535
return TLSSocketWrapper::do_handshake();
3636
}
3737

38+
TLSSocket::~TLSSocket()
39+
{
40+
/* Transport is a member of TLSSocket which is derived from TLSSocketWrapper.
41+
* Make sure that TLSSocketWrapper::close() is called before the transport is
42+
* destroyed.
43+
*/
44+
close();
45+
}
46+
3847
#endif // MBEDTLS_SSL_CLI_C

features/netsocket/TLSSocket.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,10 @@ class TLSSocket : public TLSSocketWrapper {
4141
*/
4242
TLSSocket() : TLSSocketWrapper(&tcp_socket) {}
4343

44+
/** Destroy the TLSSocket and closes the transport.
45+
*/
46+
virtual ~TLSSocket();
47+
4448
/** Create a socket on a network interface
4549
*
4650
* Creates and opens a socket on the network stack of the given

0 commit comments

Comments
 (0)