feedc0de/esp-idf
1
0
Fork 0
forked from espressif/esp-idf
Code Pull Requests Activity

wpa_supplicant: Fix some memory leak issues by coverity static analyzer.

Browse Source
This commit is contained in:
aditi_lonkar
2021-01-18 11:35:48 +05:30
parent 382fa9072d
commit 8668c11a12
2 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
components/wpa_supplicant/src/common/sae.c
View File

@@ -675,7 +675,7 @@ static int sae_derive_commit_element_ffc(struct sae_data *sae,
static int sae_derive_commit(struct sae_data *sae) static int sae_derive_commit(struct sae_data *sae)
{ {
struct crypto_bignum *mask; struct crypto_bignum *mask = NULL;
int ret = -1; int ret = -1;
unsigned int counter = 0; unsigned int counter = 0;
@@ -690,6 +690,9 @@ static int sae_derive_commit(struct sae_data *sae)
*/ */
return ESP_FAIL; return ESP_FAIL;
} }
if(mask) {
crypto_bignum_deinit(mask, 1);
}
mask = sae_get_rand_and_mask(sae); mask = sae_get_rand_and_mask(sae);
if (mask == NULL) { if (mask == NULL) {

1
components/wpa_supplicant/src/esp_supplicant/esp_wpa2.c
View File

@@ -749,6 +749,7 @@ static int eap_peer_sm_init(void)
s_wpa2_data_lock = xSemaphoreCreateRecursiveMutex(); s_wpa2_data_lock = xSemaphoreCreateRecursiveMutex();
if (!s_wpa2_data_lock) { if (!s_wpa2_data_lock) {
free(sm);
wpa_printf(MSG_ERROR, "wpa2 eap_peer_sm_init: failed to alloc data lock"); wpa_printf(MSG_ERROR, "wpa2 eap_peer_sm_init: failed to alloc data lock");
return ESP_ERR_NO_MEM; return ESP_ERR_NO_MEM;
} }