espressif/esp-idf
1
0
Fork 1
mirror of https://github.com/espressif/esp-idf.git synced 2025-07-30 18:57:19 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'fix/esp_tls_fix_memory_leak_v4.1' into 'release/v4.1'

Browse Source 
esp-tls: Fix mem leak when global_ca_store is freed

See merge request espressif/esp-idf!12086
This commit is contained in:
Mahavir Jain
2021-01-29 20:34:11 +08:00
parent 5f7f2cc70b 6282936f19
commit 744948e189
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
components/esp-tls/esp_tls_mbedtls.c
View File

@ -532,6 +532,7 @@ esp_err_t esp_mbedtls_set_global_ca_store(const unsigned char *cacert_pem_buf, c
if (ret < 0) { if (ret < 0) {
ESP_LOGE(TAG, "mbedtls_x509_crt_parse returned -0x%x", -ret); ESP_LOGE(TAG, "mbedtls_x509_crt_parse returned -0x%x", -ret);
mbedtls_x509_crt_free(global_cacert); mbedtls_x509_crt_free(global_cacert);
free(global_cacert);
global_cacert = NULL; global_cacert = NULL;
return ESP_FAIL; return ESP_FAIL;
} else if (ret > 0) { } else if (ret > 0) {
@ -550,6 +551,7 @@ void esp_mbedtls_free_global_ca_store(void)
{ {
if (global_cacert) { if (global_cacert) {
mbedtls_x509_crt_free(global_cacert); mbedtls_x509_crt_free(global_cacert);
free(global_cacert);
global_cacert = NULL; global_cacert = NULL;
} }
} }