espressif / esp-mqtt

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

MQTT_API_UNLOCK() is not called in function esp_mqtt_client_subscribe() (IDFGH-2650) #150

Closed umer-ilyas closed 4 years ago

umer-ilyas commented 4 years ago

in file mqtt_client.c at line1463 functions (esp_mqtt_client_subscribe) is returning without calling MQTT_API_UNLOCK(client);

Commit: d5e915296ebcf57d167c0410db8dd9ec746b95c5

unlock_semaphore_here.zip

david-cermak commented 4 years ago

@umer-ilyas Thanks for reporting the issue and posting the fix. Indeed this branch missing unlocking the mutex. Will be fixed soon!