- TLS: fixed memory leak when TLS authentication fails

6 years ago · c71015f240
parent 1831be663e
commit c71015f240
2 changed files with 6 additions and 1 deletions
--- a/hal/tls/mbedtls/tls_mbedtls.c
+++ b/hal/tls/mbedtls/tls_mbedtls.c
@ -388,6 +388,9 @@ TLSSocket_create(Socket socket, TLSConfiguration configuration, bool storeClient
            {
                DEBUG_PRINT("TLS", "mbedtls_ssl_handshake returned %d\n\n", ret );

+                mbedtls_ssl_config_free(&(self->conf));
+                mbedtls_ssl_free(&(self->ssl));
+
                GLOBAL_FREEMEM(self);

                return NULL;
--- a/src/mms/iso_server/iso_server.c
+++ b/src/mms/iso_server/iso_server.c
@ -521,7 +521,9 @@ handleIsoConnections(IsoServer self, bool isSingleThread)
            if (isSingleThread == false)
                IsoConnection_start(isoConnection);
        }
-
+        else {
+            Socket_destroy(connectionSocket);
+        }
    }

    if (isSingleThread)