j3d1/esp-open-rtos
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

MQTT ignores too many handlers registered

Browse source 
On MQTTClient.c function mqtt_subscribe, if more than MQTT_MAX_MESSAGE_HANDLERS are registered, the variable rc receives the value of grantedQoS.

If grantedQoS = 0, and all handlers are already occupied, it goes on and returns 0 to the caller.
This commit is contained in:
Fernando 2017-06-19 21:31:09 -03:00 committed by GitHub
parent abe3b8fd68
commit ca110fa645
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
extras/paho_mqtt_c/MQTTClient.c
View file

@ -424,6 +424,8 @@ int mqtt_subscribe(mqtt_client_t* c, const char* topic, enum mqtt_qos qos, mqtt
if (rc != 0x80) if (rc != 0x80)
{ {
int i; int i;
rc = MQTT_FAILURE;
for (i = 0; i < MQTT_MAX_MESSAGE_HANDLERS; ++i) for (i = 0; i < MQTT_MAX_MESSAGE_HANDLERS; ++i)
{ {
if (c->messageHandlers[i].topicFilter == 0) if (c->messageHandlers[i].topicFilter == 0)