Merge branch 'fix/error_codes' into 'master'

fix: regard reason codes greater than 0x80 as failures.

See merge request espressif/esp-mqtt!205
This commit is contained in:
Rocha Euripedes
2024-04-05 17:09:50 +08:00

View File

@ -1146,7 +1146,7 @@ static esp_err_t deliver_suback(esp_mqtt_client_handle_t client)
client->event.error_handle->connect_return_code = MQTT_CONNECTION_ACCEPTED;
// post data event
for (int topic = 0; topic < msg_data_len; ++topic) {
if ((uint8_t)msg_data[topic] == 0x80) {
if ((uint8_t)msg_data[topic] >= 0x80) {
client->event.error_handle->error_type = MQTT_ERROR_TYPE_SUBSCRIBE_FAILED;
break;
}