[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lwip-devel] [bug #57734] MQTT broken with muti-segments data
|
From: |
David Bourgeois |
|
Subject: |
[lwip-devel] [bug #57734] MQTT broken with muti-segments data |
|
Date: |
Sun, 16 Feb 2020 15:53:10 -0500 (EST) |
|
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.17 Safari/537.36 |
Follow-up Comment #6, bug #57734 (project lwip):
The topic is null-terminated, see mqtt.c line 755. So I think we should keep
the last null byte in the test for the topic. I also tried without without
checking the last nul byte and it also fails so the problem isn't there.
For the payload, I indeed didn't include the '\0' in the payload (rx_data) so
you're right, this is corrected now in the new patch file.
(file #48422)
_______________________________________________________
Additional Item Attachment:
File name: 0001-Add-3-tests-of-incoming-publish_v2.patch Size:9 KB
<https://savannah.nongnu.org/file/0001-Add-3-tests-of-incoming-publish_v2.patch?file_id=48422>
_______________________________________________________
Reply to this item at:
<https://savannah.nongnu.org/bugs/?57734>
_______________________________________________
Message sent via Savannah
https://savannah.nongnu.org/