buglwIP - A Lightweight TCP/IP stack - Bugs: bug #50088, socket/netconn: data before RST...

 
 

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

bug #50088: socket/netconn: data before RST should be readable

Submitter:  Simon Goldschmidt <goldsimon>
Submitted:  Thu 19 Jan 2017 10:02:14 AM UTC
   
 
Category:  sockets/netconn Severity:  3 - Normal
Item Group:  Feature Request Status:  Fixed
Privacy:  Public Assigned to:  goldsimon
Open/Closed:  Closed Planned Release:  2.1.0
lwIP version:  git head

Fri 04 Aug 2017 04:06:48 PM UTC, comment #5: 

Hi Simon,

Thanks a million. I've just tested this out and it works fine.

Regards,
Tim

Tim Cussins <tcussins>
Thu 03 Aug 2017 08:29:32 PM UTC, comment #4: 

This should be be finally fixed now.

Simon Goldschmidt <goldsimon>
Group administrator
Tue 30 May 2017 09:42:25 AM UTC, comment #3: 


> Is this what you'd expect?


No. In my tests, this worked, but as there are no continuous tests on this, maybe it's broken.

Also, I haven't tested 2 RST...

I'll reopen this to check/fix it.

Simon Goldschmidt <goldsimon>
Group administrator
Tue 30 May 2017 09:33:53 AM UTC, comment #2: 

Hi Simon,

I got a git head version of LWiP working, and have just tried this out.

With a bunch of LWiP debugging enabled, I can see the server's data arriving, then 2 further packets with RST set.

When I call lwip_recv(), I get ECONNRESET, rather than any buffered bytes.

Is this what you'd expect? Should I be enabling a particular complile-time option to get different behaviour, or using some other technique?

In short: What socket-layer calls should I make in order to read the data that arrived before RST.

Thanks,
Tim

Tim Cussins <tcussins>
Wed 12 Apr 2017 10:44:27 AM UTC, comment #1: 
Simon Goldschmidt <goldsimon>
Group administrator
Thu 19 Jan 2017 10:02:14 AM UTC, original submission:  


Simon Goldschmidt <goldsimon>
Group administrator

 

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

Attached Files
file #40815:  minicom.cap added by tcussins (5KiB - application/vnd.tcpdump.pcap - lwip debug trace of double RST and subsequent recv failure)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by tcussins (Posted a comment)
  • -email is unavailable- added by goldsimon (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 9 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-08-03 goldsimon StatusIn Progress Fixed
        Open/ClosedOpen Closed
    2017-05-30 tcussins Attached File- Added minicom.cap, #40815
    2017-05-30 goldsimon StatusFixed In Progress
        Open/ClosedClosed Open
    2017-04-12 goldsimon StatusNone Fixed
        Assigned toNone goldsimon
        Open/ClosedOpen Closed
    2017-03-01 goldsimon Planned Release2.0.2 2.1.0

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code