[lwip-devel] [bug #57734] MQTT broken with muti-segments data

[David Bourgeois]

Follow-up Comment #4, bug #57734 (project lwip):

I could reproduce the problem, it comes when the topic ends-up being spread
over 2 TCP segments. So I added 3 unit tests in the patch attached, a standard
single segment incoming publish, and 2 segmented either in the topic or
message.

The tests do pass with commit #ed561a5 :


Running suite(s): IPv4
 
  < Receive publish
    < Topic: "topic" (message len: 62)
    < Data: "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"
(len: 62, rem: 0)
> Receive segmented topic
    < Topic: "topic" (message len: 62)
    < Data: "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"
(len: 62, rem: 0)
> Receive segmented message
    < Topic: "topic" (message len: 62)
    < Data: "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"
(len: 62, rem: 0)
 SOCKETS
100%: Checks: 112, Failures: 0, Errors: 0


- They fail with master:

  < Receive publish
    < Topic: "topic" (message len: 62)
    < Data: "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"
(len: 62, rem: 0)
> Receive segmented topic
 SOCKETS
99%: Checks: 112, Failures: 0, Errors: 1
../../../../src/../test/unit/mqtt/test_mqtt.c:317:E:MQTT:segmented_sub_pub:0:
(after this point) Received signal 11 (Segmentation fault)



(file #48421)
    _______________________________________________________

Additional Item Attachment:

File name: 0001-Add-3-tests-of-incoming-publish.patch Size:9 KB
   
<https://savannah.nongnu.org/file/0001-Add-3-tests-of-incoming-publish.patch?file_id=48421>



    _______________________________________________________

Reply to this item at:

  <https://savannah.nongnu.org/bugs/?57734>

_______________________________________________
  Message sent via Savannah
  https://savannah.nongnu.org/