search

espressif / esp-mqtt

ESP32 mqtt component
Apache License 2.0
611 stars 257 forks source link

Free certs (IDFGH-10667) #264

Closed yanmartins closed 1 year ago

yanmartins commented 1 year ago

In the esp_mqtt_destroy_config function, MQTT certificates are not freed.

euripedesrocha commented 1 year ago

Hi @yanmartins, thanks for your contribution. Could you please edit your commit message with a better description of the changes?

euripedesrocha commented 1 year ago

@yanmartins, on a second look while evaluating the tests for this change I realized that we don't take ownership of this data, hence this shouldn't be freed here. The lifetime should be managed by the application and not by the mqtt client.