lwIP - A Lightweight TCP/IP stack - Bugs: bug #55607, MQTT assert when length of...
You are not allowed to post comments on this tracker with your current authentication level.
bug #55607: MQTT assert when length of received message > MQTT_VAR_HEADER_BUFFER_LEN
Submitter: | David GIRAULT <dgirault> | ||
Submitted: | Wed 30 Jan 2019 10:16:41 AM UTC | ||
Category: | apps | Severity: | 3 - Normal |
Item Group: | Crash Error | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | git head |
Jump to the original submission
Tue 30 Apr 2019 11:06:18 AM UTC, comment #7: |
Dirk Ziegelmeier <dziegel> |
Tue 23 Apr 2019 11:20:15 AM UTC, comment #6: Hi Simon,
|
David GIRAULT <dgirault> |
Fri 01 Feb 2019 02:25:33 PM UTC, comment #5: No problem, I'll do that. |
David GIRAULT <dgirault> |
Wed 30 Jan 2019 06:18:37 PM UTC, comment #4: I'd apply it, but that doesn't work. Can you pull and rebase 0002? |
Simon Goldschmidt <goldsimon> |
Wed 30 Jan 2019 03:31:09 PM UTC, comment #3: Arg, I didn't see that you had pushed this bugfix when a added the comment #2.
|
David GIRAULT <dgirault> |
Wed 30 Jan 2019 03:03:54 PM UTC, comment #2: Seen another problem with mqtt_parse_incoming() that may not call mqtt_message_received() if remaining part of current buffer isn't equal to buffer_space!
|
David GIRAULT <dgirault> |
Wed 30 Jan 2019 01:31:53 PM UTC, comment #1: Pushed, thanks for the patch! |
Simon Goldschmidt <goldsimon> |
Wed 30 Jan 2019 10:16:41 AM UTC, original submission:
The following assert in mqtt.c line 684, added by commit eea95459c9b0f150e9b9ab24e80e27e09c2612e5, completely broke MQTT reception.
|
David GIRAULT <dgirault> |
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 11 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2019-04-30 | dziegel | Status | In Progress | Fixed | |
Open/Closed | Open | Closed | |||
2019-04-23 | dgirault | Attached File | - | Added 0002-mqtt-fix-first-packet-checking-which-fail-if-MQTT_VA.patch, #46817 | |
2019-04-23 | dgirault | Attached File | - | Added 0001-mqtt-support-mostly-zero-copy-message-analysis.patch, #46816 | |
2019-01-30 | goldsimon | Status | Fixed | In Progress | |
Open/Closed | Closed | Open | |||
2019-01-30 | dgirault | Attached File | - | Added 0002-mqtt-support-mostly-zero-copy-message-analysis.patch, #46134 | |
2019-01-30 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2019-01-30 | dgirault | Attached File | - | Added 0001-mqtt-remove-bad-assert-in-mqtt_message_received.patch, #46133 |
Powered by Savane 3.13-02a9.
Corresponding source code
Applied, thanks!