From 001dc1a95eace15d543db6a23240d60e09a9f20e Mon Sep 17 00:00:00 2001 From: David Cermak Date: Thu, 11 Feb 2021 08:13:37 +0100 Subject: [PATCH] Config: Add missing nullcheck after cfg password allocation Partially addresses https://github.com/espressif/esp-idf/issues/6440 --- mqtt_client.c | 1 + 1 file changed, 1 insertion(+) diff --git a/mqtt_client.c b/mqtt_client.c index 6e74aba..42fc614 100644 --- a/mqtt_client.c +++ b/mqtt_client.c @@ -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) { client->config->clientkey_password_len = 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); }