espressif/esp-mqtt
1
0
Fork 1
mirror of https://github.com/espressif/esp-mqtt.git synced 2025-07-31 19:25:14 +02:00
Code Issues Packages Projects Releases Wiki Activity

FIX: mqtt_get_id wrong logic on msg size checks.

Browse Source 
This error was preventing the function from returning the right msg ID,
returning always 0.
This commit is contained in:
João Cabral
2019-04-18 14:32:01 +01:00
committed by David Cermak
parent d159bf4575
commit db71c753aa
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/mqtt_msg.c
View File

@@ -307,13 +307,13 @@ uint16_t mqtt_get_id(uint8_t* buffer, uint16_t length)
topiclen = buffer[i++] << 8; topiclen = buffer[i++] << 8;
topiclen |= buffer[i++]; topiclen |= buffer[i++];
if (i + topiclen >= length) if (i + topiclen > length)
return 0; return 0;
i += topiclen; i += topiclen;
if (mqtt_get_qos(buffer) > 0) if (mqtt_get_qos(buffer) > 0)
{ {
if (i + 2 >= length) if (i + 2 > length)
return 0; return 0;
//i += 2; //i += 2;
} else { } else {