From 6282936f191dccfad3ef90467bfb6b8306f14f0c 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 c187159e42..82a4f67d70 100644 --- a/components/esp-tls/esp_tls_mbedtls.c +++ b/components/esp-tls/esp_tls_mbedtls.c @@ -531,6 +531,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) { @@ -549,6 +550,7 @@ void esp_mbedtls_free_global_ca_store(void) { if (global_cacert) { mbedtls_x509_crt_free(global_cacert); + free(global_cacert); global_cacert = NULL; } }