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

Config: Add missing nullcheck after cfg password allocation

Browse Source 
Partially addresses https://github.com/espressif/esp-idf/issues/6440
This commit is contained in:
David Cermak
2021-02-11 08:13:37 +01:00
parent a5a60752c0
commit 001dc1a95e
1 changed files with 1 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
mqtt_client.c
View File

@ -475,6 +475,7 @@ esp_err_t esp_mqtt_set_config(esp_mqtt_client_handle_t client, const esp_mqtt_cl
if (config->clientkey_password && config->clientkey_password_len) { if (config->clientkey_password && config->clientkey_password_len) {
client->config->clientkey_password_len = config->clientkey_password_len; client->config->clientkey_password_len = config->clientkey_password_len;
client->config->clientkey_password = malloc(client->config->clientkey_password_len); client->config->clientkey_password = malloc(client->config->clientkey_password_len);
ESP_MEM_CHECK(TAG, client->config->clientkey_password, goto _mqtt_set_config_failed);
memcpy(client->config->clientkey_password, config->clientkey_password, client->config->clientkey_password_len); memcpy(client->config->clientkey_password, config->clientkey_password, client->config->clientkey_password_len);
} }