diff options
author | Flavio Santes <flavio.santes@1byt3.com> | 2017-11-08 22:55:30 -0500 |
---|---|---|
committer | Stig Bjørlykke <stig@bjorlykke.org> | 2017-11-09 08:45:04 +0000 |
commit | 43678551636f9845da4d52f48f8e82931f6b1c80 (patch) | |
tree | 0568e5a469b3fe62f798be4ea8042b0387898734 /summary.h | |
parent | afb252355bebee80673ea904c6b510aeaf10feb5 (diff) |
dissector/mqtt: Rewrite the SUBSCRIBE payload size computation
- FIX: subtract the property length from the message length
- Replace the 'for' loop by a 'while' loop, now that the
arithmetic is done before.
TODO: It's a protocol error (v5.0)/violation (v3.1.1) not to
include the payload in the SUBSCRIBE control packet. It would
be nice to display a "malformed packet legend" in such that
case.
Change-Id: I99ef3862aa19b3a31ea03d1c194e54f489674115
Signed-off-by: Flavio Santes <flavio.santes@1byt3.com>
Reviewed-on: https://code.wireshark.org/review/24313
Petri-Dish: Stig Bjørlykke <stig@bjorlykke.org>
Tested-by: Petri Dish Buildbot
Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
Diffstat (limited to 'summary.h')
0 files changed, 0 insertions, 0 deletions