From a5cf243ea0a84dca0640d9e77561d0ea8fe39734 Mon Sep 17 00:00:00 2001 From: Aditya Patwardhan Date: Sat, 26 Dec 2020 12:43:12 +0530 Subject: [PATCH] esp-tls: Fix mem leak when global_ca_store is freed --- components/esp-tls/esp_tls_mbedtls.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/components/esp-tls/esp_tls_mbedtls.c b/components/esp-tls/esp_tls_mbedtls.c index efedb0d70d..024da6814f 100644 --- a/components/esp-tls/esp_tls_mbedtls.c +++ b/components/esp-tls/esp_tls_mbedtls.c @@ -578,6 +578,7 @@ esp_err_t esp_mbedtls_set_global_ca_store(const unsigned char *cacert_pem_buf, c if (ret < 0) { ESP_LOGE(TAG, "mbedtls_x509_crt_parse returned -0x%x", -ret); mbedtls_x509_crt_free(global_cacert); + free(global_cacert); global_cacert = NULL; return ESP_FAIL; } else if (ret > 0) { @@ -596,6 +597,7 @@ void esp_mbedtls_free_global_ca_store(void) { if (global_cacert) { mbedtls_x509_crt_free(global_cacert); + free(global_cacert); global_cacert = NULL; } }