search

espressif / esp-mqtt

ESP32 mqtt component
Apache License 2.0
603 stars 255 forks source link

Unbalance lock in esp_mqtt_client_stop error path (IDFGH-4226) #173

Closed AxelLin closed 3 years ago

AxelLin commented 3 years ago

The commit 5e17dcaeb257f847386478119935ecd360f47377 missed an MQTT_API_UNLOCK call before return ESP_FAIL so it returns error with the lock held.

AxelLin commented 3 years ago

I sent the fix in Mqtt fixes #174

david-cermak commented 3 years ago

Fixed via https://github.com/espressif/esp-mqtt/commit/845c2a3a1e112af5dbe5f3a9ee8f6adb92a03757 Thank you @AxelLin !