espressif/esp-mqtt
1
0
Fork 1
mirror of https://github.com/espressif/esp-mqtt.git synced 2025-07-31 19:25:14 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fixed bug where the event loop wouldn't get cleaned up during destroy.

Browse Source 
The event loop would never get deleted due to the event loop handle being
cleared to 0 before checking if it exists.

Closes https://github.com/espressif/esp-idf/issues/4433
Closes IDFGH-2293
This commit is contained in:
Marius Vikhammer
2019-12-04 11:12:17 +08:00
parent 9a5187771a
commit 17e2f68e43
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
mqtt_client.c
View File

@@ -296,7 +296,6 @@ static esp_err_t esp_mqtt_destroy_config(esp_mqtt_client_handle_t client)
free(cfg->alpn_protos[i]);
}
free(cfg->alpn_protos);
memset(cfg, 0, sizeof(mqtt_config_storage_t));
free(client->connect_info.will_topic);
free(client->connect_info.will_message);
free(client->connect_info.client_id);
@@ -308,6 +307,7 @@ static esp_err_t esp_mqtt_destroy_config(esp_mqtt_client_handle_t client)
esp_event_loop_delete(client->config->event_loop_handle);
}
#endif
memset(cfg, 0, sizeof(mqtt_config_storage_t));
free(client->config);
return ESP_OK;
}