From 4df95abb22c8f90a5e7196c7c453fbbd11aa9dee Mon Sep 17 00:00:00 2001 From: Aditya Patwardhan Date: Thu, 21 Jan 2021 11:31:52 +0530 Subject: [PATCH] esp-tls: Fix mem leak when global_ca_store is freed --- components/esp-tls/esp_tls.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/components/esp-tls/esp_tls.c b/components/esp-tls/esp_tls.c index c523238810..462c5f703e 100644 --- a/components/esp-tls/esp_tls.c +++ b/components/esp-tls/esp_tls.c @@ -193,6 +193,7 @@ esp_err_t esp_tls_set_global_ca_store(const unsigned char *cacert_pem_buf, const 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) { @@ -211,6 +212,7 @@ void esp_tls_free_global_ca_store() { if (global_cacert) { mbedtls_x509_crt_free(global_cacert); + free(global_cacert); global_cacert = NULL; } }