xHasKx / luamqtt

luamqtt - Pure-lua MQTT v3.1.1 and v5.0 client
https://xhaskx.github.io/luamqtt/
MIT License
160 stars 42 forks source link

Broken contract in v5 parser? #25

Closed dvv closed 4 years ago

dvv commented 4 years ago

https://github.com/xHasKx/luamqtt/blob/master/mqtt/protocol5.lua#L729 -- here len could get false breaking the comparison with number below and dumping the program. Please revisit this chunk of code. TIA

xHasKx commented 4 years ago

Hi @dvv , thanks for the note, fixed in https://github.com/xHasKx/luamqtt/commit/bd226c0b1a743477e4099f7d270eed9d5bbb493a and bumped version to v3.3.2

dvv commented 4 years ago

Thank you!