mqtt_udp icon indicating copy to clipboard operation
mqtt_udp copied to clipboard

Published 20 hours ago • verified publisher icon dzavalishin

Some impls have wrong impl of total length encoding for lengths > 127

Open dzavalishin opened this issue 4 years ago • 1 comments

Reported (email) by Daniel Braniss

What is wrong / not implemented

For len=224 "E0 01" is sent, which is correct, and decoding side does "((E0 & ~80) << 7) + 1"

How should it be

1 << 7 | (e0 & ~80)

i.e. order of processing is wrong.

dzavalishin avatar Dec 13 '20 21:12 dzavalishin

@dzavalishin where should we add this change?

murugesanp avatar May 26 '21 07:05 murugesanp