blackbeard420/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf synced 2025-03-13 19:19:10 -04:00
Code Issues Projects Releases Wiki Activity

fix: fixed allocating if_name in client context multiple times

Browse Source 
Do not allocate client->if_name twice in esp_http_client_init().

Signed-off-by: Harshit Malpani <harshit.malpani@espressif.com>

Closes https://github.com/espressif/esp-idf/pull/13945
This commit is contained in:
fbp2m 2024-06-10 21:52:53 +02:00 committed by Aditya Patwardhan
parent 44f58ecb51
commit 1506144d3a
No known key found for this signature in database
GPG Key ID: E628B2648FBF0DD8
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
components/esp_http_client/esp_http_client.c
View File

@ -657,9 +657,11 @@ static bool init_common_tcp_transport(esp_http_client_handle_t client, const esp
}
if (config->if_name) {
client->if_name = calloc(1, sizeof(struct ifreq));
ESP_RETURN_ON_FALSE(client->if_name, false, TAG, "Memory exhausted");
memcpy(client->if_name, config->if_name, sizeof(struct ifreq));
if (client->if_name == NULL) {
client->if_name = calloc(1, sizeof(struct ifreq));
ESP_RETURN_ON_FALSE(client->if_name, false, TAG, "Memory exhausted");
memcpy(client->if_name, config->if_name, sizeof(struct ifreq));
}
esp_transport_tcp_set_interface_name(transport, client->if_name);
}
return true;