patchlwIP - A Lightweight TCP/IP stack - Patches: patch #9710, Fix mqtt unit test broken...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #9710: Fix mqtt unit test broken ebb0dc14a733

Submitter:  Axel Lin <axellin>
Submitted:  Sun 28 Oct 2018 12:01:36 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Closed
Planned Release:  2.1.1

Tue 30 Oct 2018 01:23:43 PM UTC, comment #3: 

Ok, assume this is an ack. applied and pushed.

Axel Lin <axellin>
Group Member
Tue 30 Oct 2018 11:18:54 AM UTC, comment #2: 

Can you push this?

Simon Goldschmidt <goldsimon>
Group administrator
Sun 28 Oct 2018 12:28:52 PM UTC, comment #1: 

Good catch, I don't run the unit tests on that branch too often :-(

Simon Goldschmidt <goldsimon>
Group administrator
Sun 28 Oct 2018 12:01:36 PM UTC, original submission:  

This patch is against STABLE-2_1_x branch.
Backported from b406a5438950b5b6345a9b85b6e82f36f36cdfe8, only update the subject line
because the commit id is different for STABLE-2_1_x branch.

Fixes below failure with STABLE-2_1_x branch

$ ./lwip_unittests
Running suite(s): IPv4
 IPv6
 UDP
 TCP
 TCP_OOS
 DEF
 MEM
 NETIF
 PBUF
 TIMERS
 ETHARP
 DHCP
 MDNS
 MQTT
Assertion "tcp_recved: len overflowed TCP_WND_MAX" failed at line 996 in ../../../../lwip/src/core/tcp.c
 SOCKETS
99%: Checks: 108, Failures: 0, Errors: 1
../../../../lwip/src/../test/unit/mqtt/test_mqtt.c:95:E:MQTT:basic_connect:0: (after this point) Received signal 6 (Aborted)

Axel Lin <axellin>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by goldsimon (Posted a comment)
  • -email is unavailable- added by axellin (Submitted the item)
  •  

    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 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-10-30 axellin Open/ClosedOpen Closed
    2018-10-28 axellin Attached File- Added 0001-Fix-mqtt-unit-test-broken-ebb0dc14a733.patch, #45316

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code